On 7/31/20, Harry Mallon <harry.mallon@codex.online> wrote: > Hi, > > V2 fixes all the fate tests (I didn't know to download the test footage > before. Doh). > > The following patch adds reading colour metadata (transfer, primaries and > colour space) from > standard MXF files. I wrote it without seeing the other patch but it turned > out very similar to this > one that was never merged. > > https://patchwork.ffmpeg.org/project/ffmpeg/patch/1474694344-31167-1-git-send-email-ste...@strobe.cc/ > . > > Thanks. > Harry > > From d16966e6b0a6367a5e5445987c0449fb1150785a Mon Sep 17 00:00:00 2001 > > From: Harry Mallon <harry.mallon@codex.online> > Date: Mon, 27 Jul 2020 15:52:17 +0100 > Subject: [PATCH] avformat/mxfdec: Read color metadata from MXF > > Reads color_primaries, color_trc and color_space from mxf > headers. ULs are from https://registry.smpte-ra.org/ site. > > Signed-off-by: Harry Mallon <harry.mallon@codex.online> > --- > libavformat/mxf.c | 49 +++++++++++++++++++++++++ > libavformat/mxf.h | 3 ++ > libavformat/mxfdec.c | 15 ++++++++ > tests/ref/fate/mxf-d10-user-comments | 2 +- > tests/ref/fate/mxf-opatom-user-comments | 2 +- > tests/ref/fate/mxf-probe-d10 | 2 +- > tests/ref/fate/mxf-probe-dnxhd | 4 +- > tests/ref/fate/mxf-probe-dv25 | 2 +- > tests/ref/fate/mxf-reel_name | 2 +- > tests/ref/fate/mxf-user-comments | 2 +- > 10 files changed, 75 insertions(+), 8 deletions(-) >
LGTM, but better wait for actual maintainer(s). _______________________________________________ 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".
