[FFmpeg-devel] [PATCH]Fix Pegasus lossless JPEG
Carl Eugen Hoyos
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.
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...
Size: 1324 bytes
Desc: not available
More information about the ffmpeg-devel