[Ffmpeg-devel] [PATCH] mjpeg decoder, mplayer mf://*.jpg

[Michael Niedermayer]

Hi

On Thursday 21 July 2005 11:17, Reimar D?ffinger wrote:
> Hi,
>
> On Tue, Jul 19, 2005 at 01:36:34PM +0200, Reimar D?ffinger wrote:
> > using
> > mplayer -fps 1 mf://*.jpg
> > on the files from http://www.mplayerhq.hu/MPlayer/incoming/mf_jpeg_prob
> > does not work.
> > The attached patch fixes it, though i have no idea if it is correct for
> > all MJPEG files (I guess it could be made dependant on the IJPG fourcc
> > if necessary).
>
> Still no comments? Do I have to start complaining about the not
> supported encoding variants first, so that everyone is happy that this
> is only a small patch *g*
>
> > It resets the restart_count to 0 at the beginning of every frame. I
> > think that restart_interval should probably be reset as well, but it was
> > at least not necessary for these samples.
>
> By now I'm quite sure that it is correct, since with MJPEG it should be
> possible to decode each frame independently, which wouldn't be possible
> if these values aren't reset for each frame.

well, it doesnt seem optimal, either the resting should happen after "case 
SOI:" or where you put it, but not both

[...]
-- 
Michael