[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