[FFmpeg-devel] [PATCH] mxfenc: fix essence container count value

[Matthieu Bouron]

On Thu, Oct 25, 2012 at 10:25 AM, Tomas Härdin <tomas.hardin at codemill.se>wrote:

> On Wed, 2012-10-24 at 17:14 +0200, Matthieu Bouron wrote:
> > @@ -1761,9 +1763,10 @@ static int mxf_write_header(AVFormatContext *s)
> >          present[sc->index]++;
> >      }
> >
> > -    if (s->oformat == &ff_mxf_d10_muxer) {
> > +    if (s->oformat == &ff_mxf_d10_muxer)
> >          mxf->essence_container_count = 1;
> > -    }
> > +    else if (mxf->essence_container_count > 1)
> > +        mxf->essence_container_count++;
>
> Please don't abuse this field. The EC count is the same - you just want
> to write a MultipleDescriptor ref in the Preface. I'd much prefer
> something like the attached patch.
>

Your patch looks much nicer.
Thanks !

Matthieu
[...]