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

Michael Niedermayer michaelni at gmx.at
Mon Mar 19 13:12:01 CET 2012


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

[...]

-- 
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120319/45cf4215/attachment.asc>


More information about the ffmpeg-devel mailing list