klimek added a comment. Generally, I feel like a lot of the code would be written better as ast matchers (by adding new matchers if necessary). You can use tooling::ast_matchers::matches if you want to do a match inside a callback.
================ Comment at: clang-tidy/modernize/UseDefaultCheck.cpp:278-279 @@ +277,4 @@ + +/// \brief Matcher for method declarations that are overloading the +/// copy-assignment operator. +AST_MATCHER(CXXMethodDecl, isCopyAssignmentOperator) { ---------------- FIXME: Put into ASTMatchers.h (or just do that :) http://reviews.llvm.org/D14145 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits