[FFmpeg-devel] [PATCH 01/13] avformat/mxfenc: Correct KAG alignment of preface

Michael Niedermayer michael at niedermayer.cc
Tue May 8 18:44:48 EEST 2018


On Tue, May 08, 2018 at 12:20:58PM +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 +
> >  tests/ref/fate/copy-trac4914    |  4 ++--
> >  tests/ref/fate/mxf-reel_name    |  2 +-
> >  tests/ref/fate/time_base        |  2 +-
> >  tests/ref/lavf/mxf              | 12 ++++++------
> >  tests/ref/lavf/mxf_d10          |  4 ++--
> >  tests/ref/lavf/mxf_dv25         |  4 ++--
> >  tests/ref/lavf/mxf_dvcpro50     |  4 ++--
> >  tests/ref/lavf/mxf_opatom       |  2 +-
> >  tests/ref/lavf/mxf_opatom_audio |  4 ++--
> >  10 files changed, 20 insertions(+), 19 deletions(-)
> > 
> > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> > index 3bb70326fe..c0db10b3c2 100644
> > --- a/libavformat/mxfenc.c
> > +++ b/libavformat/mxfenc.c
> > @@ -1757,6 +1757,7 @@ static int mxf_write_partition(AVFormatContext
> > *s, int bodysid,
> >          mxf_write_klv_fill(s);
> >          start = avio_tell(s->pb);
> >          mxf_write_primer_pack(s);
> > +        mxf_write_klv_fill(s);
> >          mxf_write_header_metadata_sets(s);
> >          pos = avio_tell(s->pb);
> >          header_byte_count = pos - start + klv_fill_size(pos);
> 
> Feels like such an elementary error. Probably OK, but it's been a while
> since I read the specs

will apply

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- 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/33fcd640/attachment.sig>


More information about the ffmpeg-devel mailing list