[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