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

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Tue Mar 14 00:25:41 EET 2023


James Almer:
> On 3/9/2023 11:18 AM, Raphaël Zumer wrote:
>> Hi,
>>
>> While I omitted adding v2/v3 here, I believe all comments on this set
>> of patches have been addressed so far, unless anyone strongly
>> disagrees with the rationale for moving dynamic HDR parsing and
>> serialization to libavutil or with the function signature.
>>
>> Please let me know if I missed anything.
>>
>> Thanks,
>> Raphaël Zumer
> 
> I'll apply this (and patch 1/1) in a few days if nobody comments.
> The code needs to be in lavu if we want muxers and demuxers to use this
> functionality, so moving the existing lavc functions is fine unless we
> add more avpriv_ functions that people tend to dislike.

Can we wait with this until we have the actual patches that make use of it?
(And is lavfi actually supposed to make use of this? Or why is it moved
to lavu at all? lavf can also use it in lavc.)

- Andreas



More information about the ffmpeg-devel mailing list