Replace GLenum string literals with calls to piglit_get_gl_enum_name.
---
 .../ext_texture_integer/texture-integer-glsl130.c  | 140 ++++++++++++---------
 1 file changed, 80 insertions(+), 60 deletions(-)

diff --git a/tests/spec/ext_texture_integer/texture-integer-glsl130.c 
b/tests/spec/ext_texture_integer/texture-integer-glsl130.c
index d67d588fc..224720717 100644
--- a/tests/spec/ext_texture_integer/texture-integer-glsl130.c
+++ b/tests/spec/ext_texture_integer/texture-integer-glsl130.c
@@ -48,7 +48,6 @@ static GLint BiasUniform = -1, TexUniform = -1;
 
 struct format_info
 {
-       const char *Name;
        GLenum IntFormat, BaseFormat;
        GLuint BitsPerChannel;
        GLboolean Signed;
@@ -56,80 +55,80 @@ struct format_info
 
 static const struct format_info Formats[] = {
        /*   { "GL_RGBA", GL_RGBA, GL_RGBA, 8, GL_FALSE },*/
-       { "GL_RGBA8I_EXT",   GL_RGBA8I_EXT,   GL_RGBA_INTEGER_EXT, 8,  GL_TRUE  
},
-       { "GL_RGBA8UI_EXT",  GL_RGBA8UI_EXT , GL_RGBA_INTEGER_EXT, 8,  GL_FALSE 
},
-       { "GL_RGBA16I_EXT",  GL_RGBA16I_EXT,  GL_RGBA_INTEGER_EXT, 16, GL_TRUE  
},
-       { "GL_RGBA16UI_EXT", GL_RGBA16UI_EXT, GL_RGBA_INTEGER_EXT, 16, GL_FALSE 
},
-       { "GL_RGBA32I_EXT",  GL_RGBA32I_EXT,  GL_RGBA_INTEGER_EXT, 32, GL_TRUE  
},
-       { "GL_RGBA32UI_EXT", GL_RGBA32UI_EXT, GL_RGBA_INTEGER_EXT, 32, GL_FALSE 
},
+       { GL_RGBA8I_EXT,   GL_RGBA_INTEGER_EXT, 8,  GL_TRUE  },
+       { GL_RGBA8UI_EXT , GL_RGBA_INTEGER_EXT, 8,  GL_FALSE },
+       { GL_RGBA16I_EXT,  GL_RGBA_INTEGER_EXT, 16, GL_TRUE  },
+       { GL_RGBA16UI_EXT, GL_RGBA_INTEGER_EXT, 16, GL_FALSE },
+       { GL_RGBA32I_EXT,  GL_RGBA_INTEGER_EXT, 32, GL_TRUE  },
+       { GL_RGBA32UI_EXT, GL_RGBA_INTEGER_EXT, 32, GL_FALSE },
 
-       { "GL_RGBA8I_EXT (bgra)",   GL_RGBA8I_EXT,   GL_BGRA_INTEGER_EXT, 8,  
GL_TRUE  },
-       { "GL_RGBA8UI_EXT (bgra)",  GL_RGBA8UI_EXT , GL_BGRA_INTEGER_EXT, 8,  
GL_FALSE },
-       { "GL_RGBA16I_EXT (bgra)",  GL_RGBA16I_EXT,  GL_BGRA_INTEGER_EXT, 16, 
GL_TRUE  },
-       { "GL_RGBA16UI_EXT (bgra)", GL_RGBA16UI_EXT, GL_BGRA_INTEGER_EXT, 16, 
GL_FALSE },
-       { "GL_RGBA32I_EXT (bgra)",  GL_RGBA32I_EXT,  GL_BGRA_INTEGER_EXT, 32, 
GL_TRUE  },
-       { "GL_RGBA32UI_EXT (bgra)", GL_RGBA32UI_EXT, GL_BGRA_INTEGER_EXT, 32, 
GL_FALSE },
+       { GL_RGBA8I_EXT,   GL_BGRA_INTEGER_EXT, 8,  GL_TRUE  },
+       { GL_RGBA8UI_EXT , GL_BGRA_INTEGER_EXT, 8,  GL_FALSE },
+       { GL_RGBA16I_EXT,  GL_BGRA_INTEGER_EXT, 16, GL_TRUE  },
+       { GL_RGBA16UI_EXT, GL_BGRA_INTEGER_EXT, 16, GL_FALSE },
+       { GL_RGBA32I_EXT,  GL_BGRA_INTEGER_EXT, 32, GL_TRUE  },
+       { GL_RGBA32UI_EXT, GL_BGRA_INTEGER_EXT, 32, GL_FALSE },
 
-       { "GL_RGB8I_EXT",   GL_RGB8I_EXT,   GL_RGB_INTEGER_EXT, 8,  GL_TRUE  },
-       { "GL_RGB8UI_EXT",  GL_RGB8UI_EXT , GL_RGB_INTEGER_EXT, 8,  GL_FALSE },
-       { "GL_RGB16I_EXT",  GL_RGB16I_EXT,  GL_RGB_INTEGER_EXT, 16, GL_TRUE  },
-       { "GL_RGB16UI_EXT", GL_RGB16UI_EXT, GL_RGB_INTEGER_EXT, 16, GL_FALSE },
-       { "GL_RGB32I_EXT",  GL_RGB32I_EXT,  GL_RGB_INTEGER_EXT, 32, GL_TRUE  },
-       { "GL_RGB32UI_EXT", GL_RGB32UI_EXT, GL_RGB_INTEGER_EXT, 32, GL_FALSE },
+       { GL_RGB8I_EXT,   GL_RGB_INTEGER_EXT, 8,  GL_TRUE  },
+       { GL_RGB8UI_EXT , GL_RGB_INTEGER_EXT, 8,  GL_FALSE },
+       { GL_RGB16I_EXT,  GL_RGB_INTEGER_EXT, 16, GL_TRUE  },
+       { GL_RGB16UI_EXT, GL_RGB_INTEGER_EXT, 16, GL_FALSE },
+       { GL_RGB32I_EXT,  GL_RGB_INTEGER_EXT, 32, GL_TRUE  },
+       { GL_RGB32UI_EXT, GL_RGB_INTEGER_EXT, 32, GL_FALSE },
 
-       { "GL_ALPHA8I_EXT",   GL_ALPHA8I_EXT,   GL_ALPHA_INTEGER_EXT, 8,  
GL_TRUE  },
-       { "GL_ALPHA8UI_EXT",  GL_ALPHA8UI_EXT , GL_ALPHA_INTEGER_EXT, 8,  
GL_FALSE },
-       { "GL_ALPHA16I_EXT",  GL_ALPHA16I_EXT,  GL_ALPHA_INTEGER_EXT, 16, 
GL_TRUE  },
-       { "GL_ALPHA16UI_EXT", GL_ALPHA16UI_EXT, GL_ALPHA_INTEGER_EXT, 16, 
GL_FALSE },
-       { "GL_ALPHA32I_EXT",  GL_ALPHA32I_EXT,  GL_ALPHA_INTEGER_EXT, 32, 
GL_TRUE  },
-       { "GL_ALPHA32UI_EXT", GL_ALPHA32UI_EXT, GL_ALPHA_INTEGER_EXT, 32, 
GL_FALSE },
+       { GL_ALPHA8I_EXT,   GL_ALPHA_INTEGER_EXT, 8,  GL_TRUE  },
+       { GL_ALPHA8UI_EXT , GL_ALPHA_INTEGER_EXT, 8,  GL_FALSE },
+       { GL_ALPHA16I_EXT,  GL_ALPHA_INTEGER_EXT, 16, GL_TRUE  },
+       { GL_ALPHA16UI_EXT, GL_ALPHA_INTEGER_EXT, 16, GL_FALSE },
+       { GL_ALPHA32I_EXT,  GL_ALPHA_INTEGER_EXT, 32, GL_TRUE  },
+       { GL_ALPHA32UI_EXT, GL_ALPHA_INTEGER_EXT, 32, GL_FALSE },
 
-       { "GL_LUMINANCE8I_EXT",   GL_LUMINANCE8I_EXT,   
GL_LUMINANCE_INTEGER_EXT, 8,  GL_TRUE  },
-       { "GL_LUMINANCE8UI_EXT",  GL_LUMINANCE8UI_EXT , 
GL_LUMINANCE_INTEGER_EXT, 8,  GL_FALSE },
-       { "GL_LUMINANCE16I_EXT",  GL_LUMINANCE16I_EXT,  
GL_LUMINANCE_INTEGER_EXT, 16, GL_TRUE  },
-       { "GL_LUMINANCE16UI_EXT", GL_LUMINANCE16UI_EXT, 
GL_LUMINANCE_INTEGER_EXT, 16, GL_FALSE },
-       { "GL_LUMINANCE32I_EXT",  GL_LUMINANCE32I_EXT,  
GL_LUMINANCE_INTEGER_EXT, 32, GL_TRUE  },
-       { "GL_LUMINANCE32UI_EXT", GL_LUMINANCE32UI_EXT, 
GL_LUMINANCE_INTEGER_EXT, 32, GL_FALSE },
+       { GL_LUMINANCE8I_EXT,   GL_LUMINANCE_INTEGER_EXT, 8,  GL_TRUE  },
+       { GL_LUMINANCE8UI_EXT , GL_LUMINANCE_INTEGER_EXT, 8,  GL_FALSE },
+       { GL_LUMINANCE16I_EXT,  GL_LUMINANCE_INTEGER_EXT, 16, GL_TRUE  },
+       { GL_LUMINANCE16UI_EXT, GL_LUMINANCE_INTEGER_EXT, 16, GL_FALSE },
+       { GL_LUMINANCE32I_EXT,  GL_LUMINANCE_INTEGER_EXT, 32, GL_TRUE  },
+       { GL_LUMINANCE32UI_EXT, GL_LUMINANCE_INTEGER_EXT, 32, GL_FALSE },
 
-       { "GL_LUMINANCE_ALPHA8I_EXT",   GL_LUMINANCE_ALPHA8I_EXT,   
GL_LUMINANCE_ALPHA_INTEGER_EXT, 8,  GL_TRUE  },
-       { "GL_LUMINANCE_ALPHA8UI_EXT",  GL_LUMINANCE_ALPHA8UI_EXT , 
GL_LUMINANCE_ALPHA_INTEGER_EXT, 8,  GL_FALSE },
-       { "GL_LUMINANCE_ALPHA16I_EXT",  GL_LUMINANCE_ALPHA16I_EXT,  
GL_LUMINANCE_ALPHA_INTEGER_EXT, 16, GL_TRUE  },
-       { "GL_LUMINANCE_ALPHA16UI_EXT", GL_LUMINANCE_ALPHA16UI_EXT, 
GL_LUMINANCE_ALPHA_INTEGER_EXT, 16, GL_FALSE },
-       { "GL_LUMINANCE_ALPHA32I_EXT",  GL_LUMINANCE_ALPHA32I_EXT,  
GL_LUMINANCE_ALPHA_INTEGER_EXT, 32, GL_TRUE  },
-       { "GL_LUMINANCE_ALPHA32UI_EXT", GL_LUMINANCE_ALPHA32UI_EXT, 
GL_LUMINANCE_ALPHA_INTEGER_EXT, 32, GL_FALSE },
+       { GL_LUMINANCE_ALPHA8I_EXT,   GL_LUMINANCE_ALPHA_INTEGER_EXT, 8,  
GL_TRUE  },
+       { GL_LUMINANCE_ALPHA8UI_EXT , GL_LUMINANCE_ALPHA_INTEGER_EXT, 8,  
GL_FALSE },
+       { GL_LUMINANCE_ALPHA16I_EXT,  GL_LUMINANCE_ALPHA_INTEGER_EXT, 16, 
GL_TRUE  },
+       { GL_LUMINANCE_ALPHA16UI_EXT, GL_LUMINANCE_ALPHA_INTEGER_EXT, 16, 
GL_FALSE },
+       { GL_LUMINANCE_ALPHA32I_EXT,  GL_LUMINANCE_ALPHA_INTEGER_EXT, 32, 
GL_TRUE  },
+       { GL_LUMINANCE_ALPHA32UI_EXT, GL_LUMINANCE_ALPHA_INTEGER_EXT, 32, 
GL_FALSE },
 
-       { "GL_INTENSITY8I_EXT",   GL_INTENSITY8I_EXT,   GL_RED_INTEGER_EXT, 8,  
GL_TRUE  },
-       { "GL_INTENSITY8UI_EXT",  GL_INTENSITY8UI_EXT , GL_RED_INTEGER_EXT, 8,  
GL_FALSE },
-       { "GL_INTENSITY16I_EXT",  GL_INTENSITY16I_EXT,  GL_RED_INTEGER_EXT, 16, 
GL_TRUE  },
-       { "GL_INTENSITY16UI_EXT", GL_INTENSITY16UI_EXT, GL_RED_INTEGER_EXT, 16, 
GL_FALSE },
-       { "GL_INTENSITY32I_EXT",  GL_INTENSITY32I_EXT,  GL_RED_INTEGER_EXT, 32, 
GL_TRUE  },
-       { "GL_INTENSITY32UI_EXT", GL_INTENSITY32UI_EXT, GL_RED_INTEGER_EXT, 32, 
GL_FALSE },
+       { GL_INTENSITY8I_EXT,   GL_RED_INTEGER_EXT, 8,  GL_TRUE  },
+       { GL_INTENSITY8UI_EXT , GL_RED_INTEGER_EXT, 8,  GL_FALSE },
+       { GL_INTENSITY16I_EXT,  GL_RED_INTEGER_EXT, 16, GL_TRUE  },
+       { GL_INTENSITY16UI_EXT, GL_RED_INTEGER_EXT, 16, GL_FALSE },
+       { GL_INTENSITY32I_EXT,  GL_RED_INTEGER_EXT, 32, GL_TRUE  },
+       { GL_INTENSITY32UI_EXT, GL_RED_INTEGER_EXT, 32, GL_FALSE },
 
 };
 
 static const struct format_info rg_formats[] = {
-       { "GL_RG8I",   GL_RG8I,   GL_RG_INTEGER, 8,  GL_TRUE  },
-       { "GL_RG8UI",  GL_RG8UI , GL_RG_INTEGER, 8,  GL_FALSE },
-       { "GL_RG16I",  GL_RG16I,  GL_RG_INTEGER, 16, GL_TRUE  },
-       { "GL_RG16UI", GL_RG16UI, GL_RG_INTEGER, 16, GL_FALSE },
-       { "GL_RG32I",  GL_RG32I,  GL_RG_INTEGER, 32, GL_TRUE  },
-       { "GL_RG32UI", GL_RG32UI, GL_RG_INTEGER, 32, GL_FALSE },
-       { "GL_R8I",   GL_R8I,   GL_RED_INTEGER, 8,  GL_TRUE  },
-       { "GL_R8UI",  GL_R8UI , GL_RED_INTEGER, 8,  GL_FALSE },
-       { "GL_R16I",  GL_R16I,  GL_RED_INTEGER, 16, GL_TRUE  },
-       { "GL_R16UI", GL_R16UI, GL_RED_INTEGER, 16, GL_FALSE },
-       { "GL_R32I",  GL_R32I,  GL_RED_INTEGER, 32, GL_TRUE  },
-       { "GL_R32UI", GL_R32UI, GL_RED_INTEGER, 32, GL_FALSE },
+       { GL_RG8I,   GL_RG_INTEGER, 8,  GL_TRUE  },
+       { GL_RG8UI , GL_RG_INTEGER, 8,  GL_FALSE },
+       { GL_RG16I,  GL_RG_INTEGER, 16, GL_TRUE  },
+       { GL_RG16UI, GL_RG_INTEGER, 16, GL_FALSE },
+       { GL_RG32I,  GL_RG_INTEGER, 32, GL_TRUE  },
+       { GL_RG32UI, GL_RG_INTEGER, 32, GL_FALSE },
+       { GL_R8I,   GL_RED_INTEGER, 8,  GL_TRUE  },
+       { GL_R8UI , GL_RED_INTEGER, 8,  GL_FALSE },
+       { GL_R16I,  GL_RED_INTEGER, 16, GL_TRUE  },
+       { GL_R16UI, GL_RED_INTEGER, 16, GL_FALSE },
+       { GL_R32I,  GL_RED_INTEGER, 32, GL_TRUE  },
+       { GL_R32UI, GL_RED_INTEGER, 32, GL_FALSE },
 };
 
 /* the rgb10 formats overload the Signed TRUE/FALSE member to test
  * the _REV and non-REV component ordering.
  */
 static const struct format_info rgb10_formats[] = {
-       { "GL_RGB10_A2UI", GL_RGB10_A2UI, GL_RGBA_INTEGER_EXT, 10, GL_FALSE },
-       { "GL_RGB10_A2UI (bgra)", GL_RGB10_A2UI, GL_BGRA_INTEGER_EXT, 10, 
GL_FALSE },
-       { "GL_RGB10_A2UI (rev)", GL_RGB10_A2UI, GL_RGBA_INTEGER_EXT, 10, 
GL_TRUE },
-       { "GL_RGB10_A2UI (rev bgra)", GL_RGB10_A2UI, GL_BGRA_INTEGER_EXT, 10, 
GL_TRUE },
+       { GL_RGB10_A2UI, GL_RGBA_INTEGER_EXT, 10, GL_FALSE },
+       { GL_RGB10_A2UI, GL_BGRA_INTEGER_EXT, 10, GL_FALSE },
+       { GL_RGB10_A2UI, GL_RGBA_INTEGER_EXT, 10, GL_TRUE },
+       { GL_RGB10_A2UI, GL_BGRA_INTEGER_EXT, 10, GL_TRUE },
 };
 
 static const char *FragShaderText =
@@ -323,6 +322,26 @@ test_format(const struct format_info *info)
        GLint f;
        GLfloat temp;
 
+       const char *suffix;
+       char *name;
+       if (info->IntFormat == GL_RGB10_A2UI) {
+               if (info->BaseFormat == GL_BGRA_INTEGER) {
+                       if (info->Signed)
+                               suffix = " (rev bgra)";
+                       else
+                               suffix = " (bgra)";
+               } else {
+                       if (info->Signed)
+                               suffix = " (rev)";
+                       else
+                               suffix = "";
+               }
+       } else if (info->BaseFormat == GL_BGRA_INTEGER)
+               suffix = " (bgra)";
+       else
+               suffix = "";
+       asprintf(&name, "%s%s:\n", piglit_get_gl_enum_name(info->IntFormat), 
suffix);
+
        /* pick random texture color */
        value[0] = rand() % max;
        value[1] = rand() % max;
@@ -452,7 +471,7 @@ test_format(const struct format_info *info)
            fabsf(result[2] - expected[2]) > error ||
            fabsf(result[3] - expected[3]) > error) {
                fprintf(stderr, "%s: failure with format %s:\n",
-                       TestName, info->Name);
+                       TestName, name);
                fprintf(stderr, "  texture color = %d, %d, %d, %d\n",
                        value[0], value[1], value[2], value[3]);
                fprintf(stderr, "  expected color = %g, %g, %g, %g\n",
@@ -465,6 +484,7 @@ test_format(const struct format_info *info)
        piglit_present_results();
 
        free(buf);
+       free(name);
 
        return GL_TRUE;
 }
-- 
2.15.1

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to