On Thu, Aug 20, 2020 at 10:45 AM Michael Bradshaw <mjbs...@google.com> wrote:
> On Wed, Aug 19, 2020 at 9:23 PM zhilizhao <quinkbl...@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. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".