On 01/21/2014 02:21 PM, Marek Olšák wrote:
From: Marek Olšák <marek.ol...@amd.com>
This fixes a serious regression introduced
in 4e549ddb500cf677b6fa16d9ebdfa67cc23da097.
Cc: 9.2 10.0 <mesa-sta...@lists.freedesktop.org>
---
src/gallium/auxiliary/util/u_format.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_format.h
b/src/gallium/auxiliary/util/u_format.h
index 0fbaf4c..5f86e2d 100644
--- a/src/gallium/auxiliary/util/u_format.h
+++ b/src/gallium/auxiliary/util/u_format.h
@@ -872,6 +872,9 @@ util_format_get_component_bits(enum pipe_format format,
static INLINE enum pipe_format
util_format_srgb(enum pipe_format format)
{
+ if (util_format_is_srgb(format))
+ return format;
+
switch (format) {
case PIPE_FORMAT_L8_UNORM:
return PIPE_FORMAT_L8_SRGB;
Reviewed-by: Brian Paul <bri...@vmware.com>
This looks fine for fixing things but longer-term, maybe we should
change util_format_srgb() to return the format unchanged in the default
case, as we do for util_format_linear().
I don't know if any callers depend on the default->PIPE_FORMAT_NONE
behavior now.
-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
