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