Mesa (master): glsl/tests: add GLSL_TYPE_UINT8, GLSL_TYPE_INT8 cases to switch statements

Brian Paul brianp at kemper.freedesktop.org
Tue May 1 15:29:11 UTC 2018


Module: Mesa
Branch: master
Commit: 31ab0427a767c6c8377c00203e87bf0a03ac3247
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=31ab0427a767c6c8377c00203e87bf0a03ac3247

Author: Brian Paul <brianp at vmware.com>
Date:   Thu Apr 26 11:55:46 2018 -0600

glsl/tests: add GLSL_TYPE_UINT8, GLSL_TYPE_INT8 cases to switch statements

To silence warnings about unhandled switch values.
Untested otherwise.

v2: move the INT/UINT8 cases after the INT/UINT16 cases, per Eric.

Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/compiler/glsl/tests/uniform_initializer_utils.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/compiler/glsl/tests/uniform_initializer_utils.cpp b/src/compiler/glsl/tests/uniform_initializer_utils.cpp
index 0d7fa26752..8c00c69b29 100644
--- a/src/compiler/glsl/tests/uniform_initializer_utils.cpp
+++ b/src/compiler/glsl/tests/uniform_initializer_utils.cpp
@@ -113,6 +113,8 @@ generate_data_element(void *mem_ctx, const glsl_type *type,
       case GLSL_TYPE_FLOAT16:
       case GLSL_TYPE_UINT16:
       case GLSL_TYPE_INT16:
+      case GLSL_TYPE_UINT8:
+      case GLSL_TYPE_INT8:
 	 ASSERT_TRUE(false);
 	 break;
       }
@@ -156,6 +158,8 @@ generate_data_element(void *mem_ctx, const glsl_type *type,
       case GLSL_TYPE_FLOAT16:
       case GLSL_TYPE_UINT16:
       case GLSL_TYPE_INT16:
+      case GLSL_TYPE_UINT8:
+      case GLSL_TYPE_INT8:
 	 ASSERT_TRUE(false);
 	 break;
       }
@@ -287,6 +291,8 @@ verify_data(gl_constant_value *storage, unsigned storage_array_size,
          case GLSL_TYPE_FLOAT16:
          case GLSL_TYPE_UINT16:
          case GLSL_TYPE_INT16:
+         case GLSL_TYPE_UINT8:
+         case GLSL_TYPE_INT8:
 	    ASSERT_TRUE(false);
 	    break;
 	 }




More information about the mesa-commit mailing list