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/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
