[compiz] [PATCH] Fix imageBufferToTexture for MSBFirst platforms.

David Reveman davidr at novell.com
Thu May 31 10:38:12 PDT 2007


On Wed, 2007-05-23 at 15:37 +0200, Michel Dänzer wrote:
> Fixes icon colours on my PowerBook.
> 
> Given that the preprocessor test was reversed when the code was reorganized and
> nobody on !MSBFirst platforms complained, one code path should suffice.

Hm, I'm not sure what's going on here. I fixed the mistake from the
reorganization. If byte order doesn't actually matter for the icon data
then I think the iconToTexture function should be changed to reflect
that and not imageBufferToTexture.

> ---
>  src/texture.c |    5 -----
>  1 files changed, 0 insertions(+), 5 deletions(-)
> 
> diff --git a/src/texture.c b/src/texture.c
> index 4170c70..7021643 100644
> --- a/src/texture.c
> +++ b/src/texture.c
> @@ -164,13 +164,8 @@ imageBufferToTexture (CompScreen   *screen,
>  		      unsigned int width,
>  		      unsigned int height)
>  {
> -#if IMAGE_BYTE_ORDER == MSBFirst
> -    return imageToTexture (screen, texture, image, width, height,
> -			   GL_BGRA, GL_UNSIGNED_BYTE);
> -#else
>      return imageToTexture (screen, texture, image, width, height,
>  			   GL_BGRA, GL_UNSIGNED_INT_8_8_8_8_REV);
> -#endif
>  }
>  
>  Bool
> -- 
> 1.5.2-rc3.GIT
> 
> 
> 
-David



More information about the compiz mailing list