[Piglit] [PATCH 3/3] tests/quick.tests: return to pep8 compliance

Dylan Baker baker.dylan.c at gmail.com
Tue Jan 14 15:48:45 PST 2014


commit bab84213 exposed a behvarioral problem in GleanTest, and commit
832e498 worked around this bug by reordering code. While this was a
valid work around it wasn't pep8 compliant. Since the previous patch in
this series fixed the behvaior or GleanTest, there is not reason to keep
this change.

Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
---
 tests/quick.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/tests/quick.py b/tests/quick.py
index 607680e..939aded 100644
--- a/tests/quick.py
+++ b/tests/quick.py
@@ -1,14 +1,12 @@
 # -*- coding: utf-8 -*-
 
-# XXX: These lines must be the first statements for the GleanTest.globalParams
-# assignment to be effective.  Do NOT add any import statement before.
 from framework.gleantest import GleanTest
-GleanTest.globalParams += ["--quick"]
-
 from tests.all import profile
 
 __all__ = ['profile']
 
+GleanTest.globalParams += ["--quick"]
+
 # These take too long
 del profile.tests['shaders']['glsl-fs-inline-explosion']
 del profile.tests['shaders']['glsl-fs-unroll-explosion']
-- 
1.8.5.2



More information about the Piglit mailing list