[MPlayer-dev-eng] [PATCH] remove global variables from gif demuxer

Diego Biurrun diego at biurrun.de
Sat Jan 13 05:21:42 CET 2007


On Fri, Jan 12, 2007 at 08:28:32PM +0100, Reimar Döffinger wrote:
> On Fri, Jan 12, 2007 at 08:23:02PM +0100, Reimar D?ffinger wrote:
> > does this (untested) patch look good to you?
> 
> Sorry, attached version is better...

I like it, but this seems to conflict with the fixes by John Koleszar,
which should go in ...

> --- libmpdemux/demux_gif.c	(revision 21888)
> +++ libmpdemux/demux_gif.c	(working copy)
> @@ -118,10 +122,10 @@
>  
>      // copy the pallete
>      for (y = 0; y < 256; y++) {
> -	pallete[(y * 4) + 0] = effective_map->Colors[y].Blue;
> -	pallete[(y * 4) + 1] = effective_map->Colors[y].Green;
> -	pallete[(y * 4) + 2] = effective_map->Colors[y].Red;
> -	pallete[(y * 4) + 3] = 0;
> +	priv->palette[(y * 4) + 0] = effective_map->Colors[y].Blue;
> +	priv->palette[(y * 4) + 1] = effective_map->Colors[y].Green;
> +	priv->palette[(y * 4) + 2] = effective_map->Colors[y].Red;
> +	priv->palette[(y * 4) + 3] = 0;

Nice spelling fix you snuck in there ;)

Diego



More information about the MPlayer-dev-eng mailing list