[FFmpeg-devel] [PATCH] mxf: micro patch - mxf parsing

[Alex Zhukov]

On Mon, Mar 19, 2012 at 4:02 PM, Alex Zhukov <zhukov.alex at gmail.com> wrote:
> On Mon, Mar 19, 2012 at 2:12 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> On Fri, Mar 16, 2012 at 08:47:59PM +0200, Alex Zhukov wrote:
>>> I thought this fix was not worth creating the full on patch
>>> make fate fails on adpcm_ima_qt for me so did not test
>>>
>>> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
>>> index 3d99ea7..e2eea95 100644
>>> --- a/libavformat/mxfdec.c
>>> +++ b/libavformat/mxfdec.c
>>> @@ -1877,6 +1877,7 @@ static int mxf_read_header(AVFormatContext *s)
>>>              /* next partition pack - keep going, seek to previous
>>> partition or stop */
>>>              if(mxf_parse_handle_partition_or_eof(mxf) <= 0)
>>>                  break;
>>> +            continue;
>>>          }
>>
>> this patch seems to break fate-seek-lavf_mxf
>
> yes it does
> let me figure out why

I was trying to fix ticket https://ffmpeg.org/trac/ffmpeg/ticket/1099
Maybe mxf maintainer could take a look.
Thanks

>
>>
>> [...]
>>
>> --
>> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> Awnsering whenever a program halts or runs forever is
>> On a turing machine, in general impossible (turings halting problem).
>> On any real computer, always possible as a real computer has a finite number
>> of states N, and will either halt in less than N cycles or never halt.
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>