[FFmpeg-devel] [PATCH] Pictor/PC Paint PIC decoder

Peter Ross pross
Sun May 30 12:35:56 CEST 2010


On Sun, May 30, 2010 at 11:46:36AM +0200, Michael Karcher wrote:
> Am Sonntag, den 30.05.2010, 13:21 +1000 schrieb Peter Ross:
> > +const uint32_t ff_cga3_palette[4] = {
> > +    0x000000, 0x55FFFF, 0xFF5555, 0xFFFFFF,
> > +};
> 
> I don't see that code using this CGA palette anywhere. As it is only one
> out of 4 or even 6 possible palettes (which you all seem to handle), I
> doubt it makes much sense to include this table.

Well spotted. That shouldnt have been included, as the 'cga_mode4_index'
mapping table was added to picdec.c 

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- 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/20100530/929e2477/attachment.pgp>



More information about the ffmpeg-devel mailing list