[FFmpeg-devel] [PATCH 1/3] avcodec/avcodec: Add codec_tags array to AVCodec
Anton Khirnov
anton at khirnov.net
Wed Jan 22 18:51:09 EET 2020
Quoting Michael Niedermayer (2020-01-21 21:39:03)
>
> replacing a named constant by a litteral repeated -1 is making the code
> worse. (maintaince and understanding wise)
>
> So from the smell of this thread i dont think we will agree on a public
> API and that also wasnt suggested nor is it needed ATM.
>
> Thus i suggest we keep the codec_tags as in the original patch in the
> private part and put the CODEC_TAGS_END with a FF prefix in
> internal.h
>
> That way it can be used in the fuzzer and we can in the future decide
> if this or a different API could/should become public.
>
> is that ok with everyone?
No objections from me.
--
Anton Khirnov
More information about the ffmpeg-devel
mailing list