hokein added inline comments.
================ Comment at: clang-tidy/cppcoreguidelines/ProTypeMemberInitCheck.cpp:30 +AST_MATCHER(CXXRecordDecl, hasDefaultConstructor) { + return Node.hasDefaultConstructor(); ---------------- aaron.ballman wrote: > I think this should be a public AST matcher rather than a private one; it > seems like it would be generally useful (along with some of the other similar > has* functions). However, I think it's fine for this patch. +1, will do it in a follow-up. https://reviews.llvm.org/D25747 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits