Replace GLenum string literals with calls to piglit_get_gl_enum_name.
---
 tests/spec/arb_transform_feedback2/api-queries.c | 58 ++++++++----------------
 1 file changed, 20 insertions(+), 38 deletions(-)

diff --git a/tests/spec/arb_transform_feedback2/api-queries.c 
b/tests/spec/arb_transform_feedback2/api-queries.c
index c7cf6585d..b2147d54f 100644
--- a/tests/spec/arb_transform_feedback2/api-queries.c
+++ b/tests/spec/arb_transform_feedback2/api-queries.c
@@ -53,13 +53,17 @@ static const char vstext[] =
        "}"
        ;
 
-static bool expect_bool(const char *name, bool expect, bool value)
+static bool check_bool(const GLenum param, bool expect)
 {
+       GLboolean value;
+
+       glGetBooleanv(param, &value);
+
        if (expect != value) {
                fprintf(stderr,
                        "%s has incorrect state.\n"
                        "Got %s, expected %s.\n",
-                       name,
+                       piglit_get_gl_enum_name(param),
                        value ? "true" : "false",
                        expect ? "true" : "false");
                return false;
@@ -68,13 +72,17 @@ static bool expect_bool(const char *name, bool expect, bool 
value)
        return true;
 }
 
-static bool expect_int(const char *name, int expect, int value)
+static bool check_int(const GLenum param, int expect)
 {
+       int value;
+
+       glGetIntegerv(param, &value);
+
        if (expect != value) {
                fprintf(stderr,
                        "%s has incorrect state.\n"
                        "Got %d, expected %d.\n",
-                       name, value, expect);
+                       piglit_get_gl_enum_name(param), value, expect);
                return false;
        }
 
@@ -89,17 +97,12 @@ void piglit_init(int argc, char **argv)
        GLuint vs;
        const char *varyings[] = {"x"};
        bool pass = true;
-       GLboolean bool_value;
-       GLint int_value;
 
        piglit_require_transform_feedback();
        piglit_require_GLSL();
        piglit_require_extension("GL_ARB_transform_feedback2");
 
-       glGetIntegerv(GL_TRANSFORM_FEEDBACK_BINDING, &int_value);
-       pass = expect_int("GL_TRANSFORM_FEEDBACK_BINDING",
-                          0, int_value)
-               && pass;
+       pass = check_int(GL_TRANSFORM_FEEDBACK_BINDING, 0) && pass;
 
        /* This is all just the boot-strap work for the test.
         */
@@ -128,60 +131,39 @@ void piglit_init(int argc, char **argv)
 
        /* Verify the initial state of transform feedback object queires.
         */
-       glGetIntegerv(GL_TRANSFORM_FEEDBACK_BINDING, &int_value);
-       pass = expect_int("GL_TRANSFORM_FEEDBACK_BINDING",
-                          id, int_value)
-               && pass;
+       pass = check_int(GL_TRANSFORM_FEEDBACK_BINDING, id) && pass;
 
-       glGetBooleanv(GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED, &bool_value);
-       pass = expect_bool("GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED",
-                          false, bool_value)
-               && pass;
+       pass = check_bool(GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED, false) && pass;
 
-       glGetBooleanv(GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE, &bool_value);
-       pass = expect_bool("GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE",
-                          false, bool_value)
-               && pass;
+       pass = check_bool(GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE, false) && pass;
 
        /* Make active and verify.
         */
        glBeginTransformFeedback(GL_TRIANGLES);
        pass = piglit_check_gl_error(0) && pass;
 
-       glGetBooleanv(GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE, &bool_value);
-       pass = expect_bool("GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE",
-                          true, bool_value)
-               && pass;
+       pass = check_bool(GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE, true) && pass;
 
        /* Pause and verify.
         */
        glPauseTransformFeedback();
        pass = piglit_check_gl_error(0) && pass;
 
-       glGetBooleanv(GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED, &bool_value);
-       pass = expect_bool("GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED",
-                          true, bool_value)
-               && pass;
+       pass = check_bool(GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED, true) && pass;
 
        /* Resume and verify.
         */
        glResumeTransformFeedback();
        pass = piglit_check_gl_error(0) && pass;
 
-       glGetBooleanv(GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED, &bool_value);
-       pass = expect_bool("GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED",
-                          false, bool_value)
-               && pass;
+       pass = check_bool(GL_TRANSFORM_FEEDBACK_BUFFER_PAUSED, false) && pass;
 
        /* End and verify.
         */
        glEndTransformFeedback();
        pass = piglit_check_gl_error(0) && pass;
 
-       glGetBooleanv(GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE, &bool_value);
-       pass = expect_bool("GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE",
-                          false, bool_value)
-               && pass;
+       pass = check_bool(GL_TRANSFORM_FEEDBACK_BUFFER_ACTIVE, false) && pass;
 
        glBindTransformFeedback(GL_TRANSFORM_FEEDBACK, 0);
 
-- 
2.15.1

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

Reply via email to