[compiz] Re: compiz: Changes to 'master' (Added possibility to load textures with fixed RGBA byte ordering from header files)

[Danny Baumann]

Hi,

> I find this change pretty ugly. "Bool fixedRGBA" looks like a quick hack
> to extend the imageToTexture function with some specific functionality.
> 
> Creating a lower level function that accept a format and a type GLenum
> seems like a more appropriate solution to me.

To be honest, I'm a bit surprised about that. I asked you about adding
that kind of functionality before explaining what I'm going to do, and
you agreed to that. 
Besides that, I'm not sure if we really need such an universal loader
function. However, I can have a look into it ... do you have any
proposal on how an interface (prototype) should look like?

> However, if you're storing textures in header files. I suggest that you
> store them as png data, in which case you you don't need this
> functionality in core at all as png plugin will do byte-swapping when
> necessary.

Only textures which aren't supposed to be changed and are crucial for
the operation of a plugin should be stored in header files. In my
opinion, for those important textures we should avoid creating image
loader plugin dependencies.

Regards,

Danny