[FFmpeg-devel] [PATCH 2/2] avcodec/dynamic_hdr10_plus: don't take a GetBitContext as input argument

[Jan Ekström]

On Sun, Dec 6, 2020 at 5:44 PM James Almer <jamrial at gmail.com> wrote:
>
> Create a local one instead from a byte buffer input argument.
>
> Signed-off-by: James Almer <jamrial at gmail.com>

The actual change mentioned in the commit message is LGTM, and the
updated sanity checks seem good (since now the alternatives at that
point are either 3+3 (2094-40) or 3+4 (closed captions)), but they are
pretty separate from the actual change of this commit. So in that
sense it might make sense to just separate them into their own fixup
commit when pushing. Or at least mention the sanity check fixups in
the commit message instead of them being a silent good change.

Jan