[MPlayer-dev-eng] [PATCH] MJPEG patch of APP0/COM skip

[jw203198 at hotmail.com]

>
>> >Note, that this patch should have posted to the ffmpeg-devel list.
>> >
>> 
>> In which case Mplayer should use the released ffmeg, and not
>> include its own version!
>It uses the libavcodec CVS, read README at least...

I was responding to a suggestion that the file libavcodec/mjpeg.c
somehow has something to do with ffmpeg-devel list.

>
>Of course our releases contain a snapshot of lavc.

So what does ffmpeg-devel have to do with it then?

>
>> >> The following patch for MJPEG decoding (libavcodec/mjpeg.c)
>> >> forces the use of the segment length when skipping for
>> >> certain marker types.
>> >Do you have samples which trigger this?
>> 
>> Yes.
>Maybe you could upload it to ftp.mplayerhq.hu / MPlayer/incoming/ ?
>

Is that really necessary?

Just capture a MJPEG with, say, a Matrox Gxxx card and then
try converting it to something and see what happens.
If you are lucking mencoder will abort() in build_table() because of
the erroneous marker it picks up.

Have there been any instances of the abort() in build_table()
happening?

In any case, the code is clearly wrong - you only have to
read JPEG documents (eg. itu-11505_t81e.ps) to determine that.