[FFmpeg-devel] [PATCH v2] avformat/mxfdec: Read color metadata from MXF

[Paul B Mahol]

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).