Thanks, Mohammad
On Fri, Nov 20, 2020 at 5:44 AM Anton Khirnov <an...@khirnov.net> wrote: > Quoting Mohammad Izadi (2020-11-20 04:57:11) > > From: Mohammad Izadi <moh.iz...@gmail.com> > > > > HDR10+ is dynamic metadata (A/341 Amendment - SMPTE2094-40) that needs > to be decoded from ITU-T T.35 in HEVC bitstream. The HDR10+ is transferred > to side data packet to be used or passed through. > > --- > > The fate test file can be found here: > https://drive.google.com/file/d/1vcT0ohzpoyVFcxQN32jKIhUrBaBwZweT/view?usp=sharing > > The video file needs to be copied to fate-suite/mov/ > > > configure | 1 + > > fftools/ffprobe.c | 106 +++++++++ > > libavcodec/Makefile | 3 + > > libavcodec/avpacket.c | 1 + > > libavcodec/decode.c | 1 + > > libavcodec/dynamic_hdr10_plus.c | 223 ++++++++++++++++++ > > .../dynamic_hdr10_plus.h | 22 +- > > libavcodec/hevc_sei.c | 62 ++++- > > libavcodec/hevc_sei.h | 5 + > > libavcodec/hevcdec.c | 18 ++ > > libavcodec/packet.h | 8 + > > libavcodec/version.h | 2 +- > > libavfilter/vf_showinfo.c | 106 ++++++++- > > libavutil/Makefile | 2 - > > libavutil/hdr_dynamic_metadata.c | 47 ---- > > tests/fate/mov.mak | 3 + > > tests/ref/fate/mov-hdr10-plus-metadata | 90 +++++++ > > 17 files changed, 629 insertions(+), 71 deletions(-) > > create mode 100644 libavcodec/dynamic_hdr10_plus.c > > rename libavutil/hdr_dynamic_metadata.h => > libavcodec/dynamic_hdr10_plus.h (95%) > > You cannot move a public header like that, it breaks API. > Right reverted the change in the following patch. > Also, the patch does way too many things at once, it should be split. Some changes are asked to be added by reviewers just for being able to test it. This CL has been hanging for a long time. I think Vittorio and Ian can help on this to proceed with this CL. > > -- > Anton Khirnov > _______________________________________________ > 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". _______________________________________________ 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".
