JonasToth added inline comments.
================
Comment at: clang-tidy/misc/NonPrivateMemberVariablesInClassesCheck.cpp:22
AST_MATCHER(CXXRecordDecl, hasMethods) {
- return std::distance(Node.method_begin(), Node.method_end()) != 0;
+ for (const auto &Method : Node.methods()) {
+ if (Method->isImplicit())
----------------
maybe `return llvm::any_of(Node.methods(), [](const CXXMethodDecl /* dunno
which type this would be */& M) { return !M->isImplicit(); });`?
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56966/new/
https://reviews.llvm.org/D56966
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
