Re: [Libvisual-devel] apropos VisPalette

[Dennis S. <sy...@yo...>]

Sounds good, would you like to do it before 0.1.6 or after 0.1.6 ?
It's quite a bit of work, I suspect... :(

Also we need to have good visual_palette helper functions to allocate
colors and the such!

Thanks,
Dennis



On Thu, 2004-07-01 at 13:18 -0300, Duilio Javier Protti wrote:
> I suggest to change the _VisPalette structure from:
> 
> struct _VisPalette {
>   unsigned char r[256];
>   unsigned char g[256];
>   unsigned char b[256];
> };
> 
> to 
> 
> struct _VisPalette {
>   int ncolors;
>   VisColor *colors;
> };
> 
> where _VisColor is:
> 
> struct _VisColor {
>   unsigned char r;
>   unsigned char g;
>   unsigned char b;
>   unsigned char unused;
> };
> 
> This way the palette size is not fixed,
> the colors fit well in memory, and we have
> the same structure than SDL have, so we 
> can copy palettes with just a memcpy().
> 
> 
> Regards,
> Duilio.