[PATCH] [C++11] Use std::unique_ptr for ownership in a vector.
Ahmed Charles
acharles at outlook.com
Sat Mar 8 22:36:27 PST 2014
================
Comment at: lib/ASTMatchers/Dynamic/Marshallers.h:515
@@ -514,5 +514,3 @@
- virtual ~OverloadedMatcherDescriptor() {
- llvm::DeleteContainerPointers(Overloads);
- }
+ virtual ~OverloadedMatcherDescriptor() {}
----------------
David Blaikie wrote:
> Assuming this is virtual in a base class, just omit it in this class - take the implicit definition instead.
I suppose I was being lazy and didn't want to have to try and figure that out. How about I do a pass on this class switching from virtual to override in a subsequent commit?
http://llvm-reviews.chandlerc.com/D3016
More information about the cfe-commits
mailing list