[Piglit] [PATCH 3/4] gen_const_builtin_equal_tests.py: use enumerate instead of a flag

Dylan Baker baker.dylan.c at gmail.com
Mon May 19 14:43:54 PDT 2014


This combines the two loops into a single loop using enumerate. This
should be a little faster since we loop over the data only once, and
since it uses enumerate instead of math.

Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
---
 generated_tests/gen_const_builtin_equal_tests.py | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/generated_tests/gen_const_builtin_equal_tests.py b/generated_tests/gen_const_builtin_equal_tests.py
index 4519184..79b5a1d 100644
--- a/generated_tests/gen_const_builtin_equal_tests.py
+++ b/generated_tests/gen_const_builtin_equal_tests.py
@@ -104,22 +104,19 @@ def main():
     except OSError:
         pass
 
-    test_id = 2
-    for x in test_vectors:
-        name = "spec/glsl-1.20/execution/built-in-functions/glsl-const-builtin-equal-%02d.shader_test" % test_id
-        test_id = test_id + 1
-
+    for test_id, x in enumerate(test_vectors, 2):
+        # make equal tests
+        name = ("spec/glsl-1.20/execution/built-in-functions/"
+                "glsl-const-builtin-equal-{0:02d}.shader_test".format(test_id))
         print(name)
 
         with open(name, 'w+') as f:
             f.write(TEMPLATE.render_unicode(
                 func='equal', input=x[0:2], expected=x[2]))
 
-
-    test_id = 2
-    for x in test_vectors:
-        name = "spec/glsl-1.20/execution/built-in-functions/glsl-const-builtin-notEqual-%02d.shader_test" % test_id
-        test_id = test_id + 1
+        # make notEqual tests
+        name = ("spec/glsl-1.20/execution/built-in-functions/"
+                "glsl-const-builtin-notEqual-{0:02d}.shader_test".format(test_id))
 
         # When generating the notEqual tests, each of the values in the
         # expected result vector need to be inverted
-- 
2.0.0.rc2



More information about the Piglit mailing list