[PATCH] D14145: modernize-use-default supports copy constructor and copy-assignment operator.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 28 04:40:16 PDT 2015

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 :)


More information about the cfe-commits mailing list