[FFmpeg-devel] [PATCH]Fix palette calculation in some decoders

Michael Niedermayer michaelni at gmx.at
Fri Nov 11 23:18:03 CET 2011


On Fri, Nov 11, 2011 at 10:44:18PM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> If I am not missing something, FFmpeg currently happily assumes complete 
> transparency for most PAL8 codecs.
> 
> I did not find the palette code in vmnc.c, I am unsure if ff_cga_palette[] is 
> correct (black is always transparent?).
> 
> Please comment, I did not test the ones with "AV_RB24()" and 
> "bytestream_get_be24()".
> 
> Thank you, Carl Eugen

The patch looks good but it should be split to make future bisect
easier,
also i see several unrelated bugfixes in there, they too should
be split out

otherwise LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111111/64ea4ade/attachment.asc>


More information about the ffmpeg-devel mailing list