[PATCH] D17253: Cleanup of analyzer scripts as suggested by pychecker and pep8

Alexander Riccio via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 15 14:19:42 PST 2016


ariccio added inline comments.

================
Comment at: C:/LLVM/llvm/tools/clang/utils/analyzer/CmpRuns.py:194
@@ -182,3 +193,3 @@
 # Backward compatibility API.
-def loadResults(path, opts, root = "", deleteEmpty=True):
+def loadResults(path, opts, root="", deleteEmpty=True):
     return loadResultsFromSingleRun(SingleRunInfo(path, root, opts.verboseLog),
----------------
zaks.anna wrote:
> PEP 0008: Don't use spaces around the = sign when used to indicate a keyword argument or a default parameter value.
> 
> 
...agreed? Huh?


http://reviews.llvm.org/D17253





More information about the cfe-commits mailing list