[FFmpeg-devel] [PATCH] avcodec/proresenc: add support for PQ and HLG

Michael Bradshaw mjbshaw at google.com
Fri Aug 21 21:10:53 EEST 2020


On Thu, Aug 20, 2020 at 10:45 AM Michael Bradshaw <mjbshaw at google.com>
wrote:

> On Wed, Aug 19, 2020 at 9:23 PM zhilizhao <quinkblack at foxmail.com> wrote:
>
>> > -static const int valid_trc[4]        = { AVCOL_TRC_RESERVED0,
>> AVCOL_TRC_BT709, AVCOL_TRC_UNSPECIFIED, INT_MAX };
>> > +static const int valid_trc[6]        = { AVCOL_TRC_RESERVED0,
>> AVCOL_TRC_BT709, AVCOL_TRC_UNSPECIFIED, AVCOL_TRC_SMPTE2084,
>> > +                                         AVCOL_TRC_ARIB_STD_B67,
>> INT_MAX };
>>
>> For future-proof, how about remove the array size?
>
>
> Sure, but maybe as a follow-up patch? I'd like to remove the array size
> from all of the arrays, but I don't want this particular patch to grow
> beyond its original scope.
>

Pushed this patch. I sent out another patch to remove the array size and
instead infer it.


More information about the ffmpeg-devel mailing list