[Piglit] [PATCH] ext_timer_query: the timestamp test does not require the EXT extension

Emil Velikov emil.l.velikov at gmail.com
Wed Dec 3 05:37:27 PST 2014


The test itself is written against the ARB extension, and neither the
test or the ARB extension requires the EXT one.

Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
---

It's unlikely that anyone will hit this (i.e. has support for ARB but 
lacks the EXT extension) but from a quick look at the spec it seems 
like the correct thing to do.

-Emil

 tests/spec/ext_timer_query/time-elapsed.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/spec/ext_timer_query/time-elapsed.c b/tests/spec/ext_timer_query/time-elapsed.c
index 3085c67..65a9301 100644
--- a/tests/spec/ext_timer_query/time-elapsed.c
+++ b/tests/spec/ext_timer_query/time-elapsed.c
@@ -292,10 +292,10 @@ piglit_init(int argc, char **argv)
 	prog = piglit_build_simple_program(vs_text, fs_text);
 	iters_loc = glGetUniformLocation(prog, "iters");
 
-	piglit_require_extension("GL_EXT_timer_query");
-
 	if (argc == 2 && strcmp(argv[1], "timestamp") == 0) {
 		piglit_require_extension("GL_ARB_timer_query");
 		test = TIMESTAMP;
+	} else {
+		piglit_require_extension("GL_EXT_timer_query");
 	}
 }
-- 
2.1.3



More information about the Piglit mailing list