[FFmpeg-devel] [PATCH v4 1/2] avcodec/avutil: move dynamic HDR10+ metadata parsing to libavutil

James Almer jamrial at gmail.com
Mon Mar 13 22:38:00 EET 2023


On 3/13/2023 5:22 PM, Raphaël Zumer wrote:
> Signed-off-by: Raphaël Zumer <rzumer at tebako.net>
> ---
>   libavcodec/Makefile              |   2 +-
>   libavcodec/dynamic_hdr10_plus.c  | 198 -------------------------------
>   libavcodec/dynamic_hdr10_plus.h  |  35 ------
>   libavcodec/h2645_sei.c           |   6 +-
>   libavutil/hdr_dynamic_metadata.c | 180 ++++++++++++++++++++++++++++
>   libavutil/hdr_dynamic_metadata.h |  11 ++
>   6 files changed, 195 insertions(+), 237 deletions(-)
>   delete mode 100644 libavcodec/dynamic_hdr10_plus.c
>   delete mode 100644 libavcodec/dynamic_hdr10_plus.h

This is missing the required changes in the av1 and libdav1d decoders 
like you did for h2645_sei. Support in them is pretty recent, so i 
assume your tree is slightly outdated? Otherwise compilation should have 
failed for you.
I can make the changes before pushing since it's a one line change for 
each decoder, so no need for a new version.


More information about the ffmpeg-devel mailing list