[Piglit] [Patch v2 1/8] igt.py: Fix whitepace warnings for pylint

Dylan Baker baker.dylan.c at gmail.com
Mon Jan 26 11:28:08 PST 2015


v2: - Also remove stderr from subprocess.Popen

Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
---
 tests/igt.py | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/tests/igt.py b/tests/igt.py
index e65b8a9..aa635c7 100644
--- a/tests/igt.py
+++ b/tests/igt.py
@@ -129,22 +129,20 @@ tests.extend(listTests("multi-tests"))
 
 def addSubTestCases(test):
     proc = subprocess.Popen(
-            [os.path.join(igtTestRoot, test), '--list-subtests'],
-            stdout=subprocess.PIPE,
-            stderr=subprocess.PIPE,
-            env=os.environ.copy(),
-            universal_newlines=True
-            )
-    out, err = proc.communicate()
+        [os.path.join(igtTestRoot, test), '--list-subtests'],
+        stdout=subprocess.PIPE,
+        env=os.environ.copy(),
+        universal_newlines=True)
+    out, _ = proc.communicate()
 
     # a return code of 79 indicates there are no subtests
     if proc.returncode == 79:
-         profile.test_list[grouptools.join('igt', test)] = IGTTest(test)
-         return
+        profile.test_list[grouptools.join('igt', test)] = IGTTest(test)
+        return
 
     if proc.returncode != 0:
-         print "Error: Could not list subtests for " + test
-         return
+        print "Error: Could not list subtests for " + test
+        return
 
     subtests = out.split("\n")
 
-- 
2.2.2



More information about the Piglit mailing list