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
