[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