mesa: fix SRGB alpha channel value in pack_float_R8G8B8X8_SRGB
authorDave Airlie <airlied@redhat.com>
Wed, 17 Sep 2014 01:56:47 +0000 (11:56 +1000)
committerDave Airlie <airlied@redhat.com>
Wed, 17 Sep 2014 03:28:18 +0000 (13:28 +1000)
Jason pointed out the bug on review adding new formats,
but the existing format also appears to have the bug, so
use 255 as the max, these are SRGB no SNORM.

Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
src/mesa/main/format_pack.c

index 6cbf8593bcafc045216333b4ab98a0c2a50395fd..6bf174a81f78f133e68709e864c270d3ae5a7c2f 100644 (file)
@@ -1713,7 +1713,7 @@ pack_float_R8G8B8X8_SRGB(const GLfloat src[4], void *dst)
    GLubyte r = util_format_linear_float_to_srgb_8unorm(src[RCOMP]);
    GLubyte g = util_format_linear_float_to_srgb_8unorm(src[GCOMP]);
    GLubyte b = util_format_linear_float_to_srgb_8unorm(src[BCOMP]);
-   *d = PACK_COLOR_8888(127, b, g, r);
+   *d = PACK_COLOR_8888(255, b, g, r);
 }