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 <[email protected]> --- 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 _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
