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

Alexander Riccio via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 27 21:12:10 PST 2016


ariccio added inline comments.

================
Comment at: C:/LLVM/llvm/tools/clang/utils/analyzer/SATestBuild.py:521
@@ -502,3 +520,3 @@
             SummaryLog.write("See the first %d below.\n"
-                                                   % (NumOfFailuresInSummary,))
+                             (NumOfFailuresInSummary,))
         # TODO: Add a line "See the results folder for more."
----------------
zaks.anna wrote:
> Is removing '%' correct here?
Nope. Good catch!


http://reviews.llvm.org/D17253





More information about the cfe-commits mailing list