[FFmpeg-devel] [PATCH]Fix Pegasus lossless JPEG

Carl Eugen Hoyos cehoyos
Wed Nov 11 22:03:00 CET 2009


On Monday 12 October 2009 01:36:56 Michael Niedermayer wrote:
> On Sun, Oct 11, 2009 at 10:27:33PM +0200, Carl Eugen Hoyos wrote:

> > -            s->avctx->pix_fmt = PIX_FMT_RGB32;
> > +            s->avctx->pix_fmt = s->avctx->codec_tag == MKTAG('J', 'P',
> > 'G', 'L') ? PIX_FMT_BGR32 : PIX_FMT_RGB32;
>
> this looks wrong (both original & new code), have you tested this on
> little & big endian?

Attached version was tested by me on little and by Diego on big endian (he 
actually tested the original version - both use PIX_FMT_BGR32).

I was not able to test "original", because I did not find a non-lossless RGB 
MJPEG sample on natsuki.

Carl Eugen

PS: Searching for my original message, I found an answer that I did not see 
originally (postponed?) - I can't comment on its content, since I don't 
exactly understand it...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patchmjpeg-2.diff
Type: text/x-diff
Size: 1324 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091111/f44931bb/attachment.diff>



More information about the ffmpeg-devel mailing list