[PATCH] Force braces on the else branch if they are being added to the if branch.

Alexander Kornienko alexfh at google.com
Tue Mar 31 02:43:14 PDT 2015


Awesome! Thanks for the fix!


================
Comment at: clang-tidy/readability/BracesAroundStatementsCheck.h:46
@@ -45,3 +45,3 @@
 private:
-  void checkStmt(const ast_matchers::MatchFinder::MatchResult &Result,
+  bool checkStmt(const ast_matchers::MatchFinder::MatchResult &Result,
                  const Stmt *S, SourceLocation StartLoc,
----------------
Maybe add a comment describing the meaning of the return value?

http://reviews.llvm.org/D8708

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list