[llvm] r283707 - [lit] Remove semicolons in Python code

Brian Gesiak via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 9 18:18:14 PDT 2016


Author: modocache
Date: Sun Oct  9 20:18:14 2016
New Revision: 283707

URL: http://llvm.org/viewvc/llvm-project?rev=283707&view=rev
Log:
[lit] Remove semicolons in Python code

Summary:
Semicolons aren't necessary as statement terminators in Python, and
each of these uses are superfluous as they appear at the end of a line.
The convention is to not use semicolons where not needed, so remove them.

Reviewers: echristo, delcypher, beanz, ddunbar

Subscribers: llvm-commits, mehdi_amini

Differential Revision: https://reviews.llvm.org/D25409

Modified:
    llvm/trunk/utils/lit/lit/TestRunner.py
    llvm/trunk/utils/lit/lit/util.py

Modified: llvm/trunk/utils/lit/lit/TestRunner.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/TestRunner.py?rev=283707&r1=283706&r2=283707&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/TestRunner.py (original)
+++ llvm/trunk/utils/lit/lit/TestRunner.py Sun Oct  9 20:18:14 2016
@@ -503,7 +503,7 @@ def executeScriptInternal(test, litConfi
     return out, err, exitCode, timeoutInfo
 
 def executeScript(test, litConfig, tmpBase, commands, cwd):
-    bashPath = litConfig.getBashPath();
+    bashPath = litConfig.getBashPath()
     isWin32CMDEXE = (litConfig.isWindows and not bashPath)
     script = tmpBase + '.script'
     if isWin32CMDEXE:

Modified: llvm/trunk/utils/lit/lit/util.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/util.py?rev=283707&r1=283706&r2=283707&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/util.py (original)
+++ llvm/trunk/utils/lit/lit/util.py Sun Oct  9 20:18:14 2016
@@ -114,8 +114,8 @@ def which(command, paths = None):
 def checkToolsPath(dir, tools):
     for tool in tools:
         if not os.path.exists(os.path.join(dir, tool)):
-            return False;
-    return True;
+            return False
+    return True
 
 def whichTools(tools, paths):
     for path in paths.split(os.pathsep):




More information about the llvm-commits mailing list