[FFmpeg-devel] [PATCH 10/13] avformat/mxfenc: Set color siting to 0 for D10-MXF
Michael Niedermayer
michael at niedermayer.cc
Tue May 8 20:41:33 EEST 2018
On Tue, May 08, 2018 at 12:55:18PM +0200, Tomas Härdin wrote:
> mån 2018-05-07 klockan 12:38 +0200 skrev Michael Niedermayer:
> > > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > ---
> > libavformat/mxfenc.c | 1 +
> > .../ref/fate/concat-demuxer-extended-lavf-mxf | 2 +-
> > .../fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +-
> > .../ref/fate/concat-demuxer-simple1-lavf-mxf | 242 +++++++++---------
> > .../fate/concat-demuxer-simple1-lavf-mxf_d10 | 140 +++++-----
> > tests/ref/seek/lavf-mxf | 44 ++--
> > tests/ref/seek/lavf-mxf_d10 | 54 ++--
> > tests/ref/seek/lavf-mxf_dv25 | 54 ++--
> > tests/ref/seek/lavf-mxf_dvcpro50 | 54 ++--
> > tests/ref/seek/lavf-mxf_opatom | 54 ++--
> > tests/ref/seek/lavf-mxf_opatom_audio | 54 ++--
> > 11 files changed, 351 insertions(+), 350 deletions(-)
> >
> > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> > index f2be76cc86..adf5527534 100644
> > --- a/libavformat/mxfenc.c
> > +++ b/libavformat/mxfenc.c
> > @@ -2361,6 +2361,7 @@ static int mxf_write_header(AVFormatContext *s)
> > mxf->edit_unit_byte_count += klv_fill_size(mxf->edit_unit_byte_count);
> >
> > sc->signal_standard = 1;
> > + sc->color_siting = 0;
> > }
>
> Can't find anything in my documents that says anything about this. I
> don't remember what D-10 is actually specified in...
SMPTE 386M (D-10) lists 4 as value to be used
SMPTE 377-1-2009 says
"The definitions of 00h (coSiting) and 04h (Rec 601) are equivalent. The value of 04h is deprecated. New
MXF encoders shall use the value of 00h instead."
I will add this to the commit message
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180508/f8e3531b/attachment.sig>
More information about the ffmpeg-devel
mailing list