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

[Manuel Klimek via cfe-commits]

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