zaks.anna requested changes to this revision.
zaks.anna added a comment.
This revision now requires changes to proceed.

Not sure how you got these changes, but some of them seem wrong and some seem 
inconsistently applied.

Has this been tested?


================
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),
----------------
PEP 0008: Don't use spaces around the = sign when used to indicate a keyword 
argument or a default parameter value.




http://reviews.llvm.org/D17253



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to