[PATCH] [clang-tidy] Assert related checkers
Alexander Kornienko
alexfh at google.com
Fri Feb 27 09:31:18 PST 2015
Looks good! (with one nit, see the comment)
Thanks for the contribution!
Do you need me to commit the patch for you?
================
Comment at: clang-tidy/misc/AssertSideEffectCheck.cpp:96
@@ +95,3 @@
+void AssertSideEffectCheck::check(const MatchFinder::MatchResult &Result) {
+ const ASTContext * ASTCtx = Result.Context;
+ const auto *CondStmt = Result.Nodes.getNodeAs<Stmt>("condStmt");
----------------
nit: Remove the space after the star.
http://reviews.llvm.org/D7375
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list