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

Reply via email to