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

[Tomas Härdin]

lör 2020-08-01 klockan 16:17 +0200 skrev 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

Pushed

/Tomas