[FFmpeg-devel] [PATCH] get rid of nonsense palette pointer for RGB8, GRAY8 etc.

Michael Niedermayer michaelni
Fri Mar 20 19:10:25 CET 2009

On Fri, Mar 20, 2009 at 06:40:57PM +0100, Reimar D?ffinger wrote:
> Hello,
> I have no idea why that code came to be like that, but I know for sure
> it causes regression tests to fail because the silly copyPlanar function
> things that palette pointer is the U plane it needs to copy.
> Probably that function should be fixed anyway, but this change still
> seems right:

pixfmt.h says:

 * For all the 8bit per pixel formats, an RGB32 palette is in data[1] like
 * for pal8. This palette is filled in automatically by the function
 * allocating the picture.

so indeed a palette should be in data[1]

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090320/69d71d12/attachment.pgp>

More information about the ffmpeg-devel mailing list