[FFmpeg-devel] [PATCH v2] avformat/mxfdec: Read color metadata from MXF
Paul B Mahol
onemda at gmail.com
Fri Jul 31 18:57:16 EEST 2020
On 7/31/20, Harry Mallon <harry.mallon at 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-steven@strobe.cc/
> .
>
> Thanks.
> Harry
>
> From d16966e6b0a6367a5e5445987c0449fb1150785a Mon Sep 17 00:00:00 2001
>
> From: Harry Mallon <harry.mallon at 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 at 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).
More information about the ffmpeg-devel
mailing list