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

Matthieu Bouron matthieu.bouron at gmail.com
Thu Oct 25 11:37:59 CEST 2012


On Thu, Oct 25, 2012 at 11:21 AM, Matthieu Bouron <matthieu.bouron at gmail.com
> wrote:

> 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 !
>

Also, don't forget to update the fate tests.


More information about the ffmpeg-devel mailing list