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

[Tomas Härdin]

fre 2020-07-31 klockan 11:09 +0100 skrev Harry Mallon:
> 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(-)

Looks OK to me.

/Tomas