[Piglit] [PATCH 07/31] transform feedback2 api query: Remove GLenum string literals.

Fabian Bieler fabianbieler at fastmail.fm
Thu Dec 28 15:57:33 UTC 2017


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



More information about the Piglit mailing list