[Piglit] [PATCH 10/31] texture integer glsl130: Remove GLenum string literals.
Fabian Bieler
fabianbieler at fastmail.fm
Thu Dec 28 15:57:36 UTC 2017
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
More information about the Piglit
mailing list