[FFmpeg-devel] [PATCH 2/2] avutil: add HDR10+ dynamic metadata serialization function

Derek Buitenhuis derek.buitenhuis at gmail.com
Thu Mar 2 22:37:10 EET 2023


On 3/2/2023 8:24 PM, Leo Izen wrote:
> Why not put this in avcodec/dynamic_hdr10_plus.c? You reference 
> put_bits.h which is in avcodec, so that can possibly be an issue, even 
> though it is inlined (i.e. it sends the wrong message since avutil is 
> supposed to not depend on avcodec).

put_bits is a header-only implementation, isn't it? No dependency is
introduced.

Putting it in avutil was my suggested, as it is a function which takes
side data as input, FWIW.

- Derek


More information about the ffmpeg-devel mailing list