Re: [PATCH] D13890: Apply modernize-use-default to clang.

2015-10-20 Thread Benjamin Kramer via cfe-commits
bkramer added a comment. feel free to submit http://reviews.llvm.org/D13890 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13890: Apply modernize-use-default to clang.

2015-10-20 Thread Angel Garcia via cfe-commits
angelgarcia updated this revision to Diff 37867. angelgarcia added a comment. Put comment back. http://reviews.llvm.org/D13890 Files: include/clang/AST/ASTConsumer.h include/clang/AST/CanonicalType.h include/clang/AST/DeclFriend.h include/clang/AST/DeclTemplate.h include/clang/AST/Exp

Re: [PATCH] D13890: Apply modernize-use-default to clang.

2015-10-20 Thread Benjamin Kramer via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. Looks good with one comment. Comment at: lib/StaticAnalyzer/Core/MemRegion.cpp:141-143 @@ -140,5 +141,1 @@ -MemRegionManager::~MemRegionManager() { - // All regions and the

Re: [PATCH] D13890: Apply modernize-use-default to clang.

2015-10-20 Thread Manuel Klimek via cfe-commits
klimek added a reviewer: bkramer. klimek added a comment. +benjamin, who has been doing those changes lately... http://reviews.llvm.org/D13890 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

[PATCH] D13890: Apply modernize-use-default to clang.

2015-10-20 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: klimek. angelgarcia added subscribers: cfe-commits, alexfh. Herald added a subscriber: klimek. Replace empty bodies of default constructors and destructors with '= default'. http://reviews.llvm.org/D13890 Files: include/clang/AST