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

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 20 05:40:45 PDT 2015


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 their data are BumpPtrAllocated.  No need to call
-  // their destructors.
-}
 
----------------
Please keep the comments.


http://reviews.llvm.org/D13890





More information about the cfe-commits mailing list