[Mesa-dev] [PATCH v2 03/23] mesa/colormac: Remove an unused macro

Ian Romanick idr at freedesktop.org
Wed Dec 3 14:21:01 PST 2014


This patch is

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 12/01/2014 03:04 AM, Iago Toral Quiroga wrote:
> From: Jason Ekstrand <jason.ekstrand at intel.com>
> 
> The PACK_565_REV macro is no longer used.  It was also extremely confusing
> because it's actually a byteswapped 565 not reversed 565.
> 
> Signed-off-by: Jason Ekstrand <jason.ekstrand at intel.com>
> ---
>  src/mesa/main/colormac.h | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/src/mesa/main/colormac.h b/src/mesa/main/colormac.h
> index c8adca6..bc69f46 100644
> --- a/src/mesa/main/colormac.h
> +++ b/src/mesa/main/colormac.h
> @@ -69,9 +69,6 @@ _mesa_unclamped_float_rgba_to_ubyte(GLubyte dst[4], const GLfloat src[4])
>  #define PACK_COLOR_565( X, Y, Z )                                  \
>     ((((X) & 0xf8) << 8) | (((Y) & 0xfc) << 3) | (((Z) & 0xf8) >> 3))
>  
> -#define PACK_COLOR_565_REV( X, Y, Z ) \
> -   (((X) & 0xf8) | ((Y) & 0xe0) >> 5 | (((Y) & 0x1c) << 11) | (((Z) & 0xf8) << 5))
> -
>  #define PACK_COLOR_5551( R, G, B, A )					\
>     ((((R) & 0xf8) << 8) | (((G) & 0xf8) << 3) | (((B) & 0xf8) >> 2) |	\
>      ((A) >> 7))
> 



More information about the mesa-dev mailing list