fixed pylint warnings

Author: dougxc

java-benchmarks/mx.java-benchmarks/mx_java_benchmarks.py

@@ -772,7 +772,7 @@ def workingDirectory(self, benchmarks, bmSuiteArgs):
     def createCommandLineArgs(self, benchmarks, bmSuiteArgs):
         if benchmarks is None:
             # No benchmark specified in the command line, so run everything.
-            benchmarks = [b for b in self.benchmarkList(bmSuiteArgs)]
+            benchmarks = self.benchmarkList(bmSuiteArgs)
 
         vmArgs = self.vmArgs(bmSuiteArgs)
         runArgs = self.runArgs(bmSuiteArgs)

sdk/mx.sdk/mx_sdk_vm_impl.py

@@ -2325,8 +2325,7 @@ def get_standalone_distribution(comp_dir_name):
             if standalone.main_comp_dir_name == comp_dir_name:
                 return standalone
         raise mx.abort("Cannot find a standalone with dir_name '{}'.\nAvailable standalones:\n{}".format(comp_dir_name, '\n'.join((('- ' + s.main_comp_dir_name for s in standalones)))))
-    else:
-        raise mx.abort('No standalones available. Did you forget to dynamically import a component?')
+    raise mx.abort('No standalones available. Did you forget to dynamically import a component?')
 
 
 def has_svm_polyglot_lib():

substratevm/mx.substratevm/mx_substratevm_benchmark.py

@@ -301,7 +301,7 @@ def extract_dacapo(dacapo_path):
         return extract_archive(dacapo_path, 'dacapo.extracted')
 
     def benchmark_resources(self, benchmark):
-        pass
+        return None
 
     def additional_lib(self, lib):
         return mx.library(lib).get_path(True)

truffle/mx.truffle/tck.py

@@ -147,8 +147,7 @@ def pull(self):
         if ret_code != 0:
             if self.required:
                 raise Abort('Cannot download artifact {0} '.format(self))
-            else:
-                return False
+            return False
         else:
             return True