diff options
| author | Ahmed Charles <ahmedcharles@gmail.com> | 2014-03-05 08:25:08 +0000 |
|---|---|---|
| committer | Ahmed Charles <ahmedcharles@gmail.com> | 2014-03-05 08:25:08 +0000 |
| commit | b5dab28db5f46fb3f0b75238b6cb60f63b5a7d4f (patch) | |
| tree | 1183ad6341759844e45bd80bd4bb03400d007332 | |
| parent | 4b3c66e6e816ad87e1c81fadf35810243426d903 (diff) | |
| download | bcm5719-llvm-b5dab28db5f46fb3f0b75238b6cb60f63b5a7d4f.tar.gz bcm5719-llvm-b5dab28db5f46fb3f0b75238b6cb60f63b5a7d4f.zip | |
[C++11] Add release() to OwningPtr.
This will make the transition to unique_ptr easier by allowing more
incremental changes.
llvm-svn: 202949
| -rw-r--r-- | llvm/include/llvm/ADT/OwningPtr.h | 2 | ||||
| -rw-r--r-- | llvm/unittests/ADT/OwningPtrTest.cpp | 18 |
2 files changed, 20 insertions, 0 deletions
diff --git a/llvm/include/llvm/ADT/OwningPtr.h b/llvm/include/llvm/ADT/OwningPtr.h index 5453d5c84e4..034bcfd6edd 100644 --- a/llvm/include/llvm/ADT/OwningPtr.h +++ b/llvm/include/llvm/ADT/OwningPtr.h @@ -73,6 +73,8 @@ public: return Tmp; } + T *release() { return take(); } + std::unique_ptr<T> take_unique() { return std::unique_ptr<T>(take()); } T &operator*() const { diff --git a/llvm/unittests/ADT/OwningPtrTest.cpp b/llvm/unittests/ADT/OwningPtrTest.cpp index aef4d97bd66..8b10f475d09 100644 --- a/llvm/unittests/ADT/OwningPtrTest.cpp +++ b/llvm/unittests/ADT/OwningPtrTest.cpp @@ -90,6 +90,24 @@ TEST_F(OwningPtrTest, Take) { EXPECT_EQ(1u, TrackDestructor::Destructions); } +TEST_F(OwningPtrTest, Release) { + TrackDestructor::ResetCounts(); + TrackDestructor *T = 0; + { + OwningPtr<TrackDestructor> O(new TrackDestructor(3)); + T = O.release(); + EXPECT_FALSE((bool)O); + EXPECT_TRUE(!O); + EXPECT_FALSE(O.get()); + EXPECT_FALSE(O.isValid()); + EXPECT_TRUE(T); + EXPECT_EQ(3, T->val); + EXPECT_EQ(0u, TrackDestructor::Destructions); + } + delete T; + EXPECT_EQ(1u, TrackDestructor::Destructions); +} + TEST_F(OwningPtrTest, MoveConstruction) { TrackDestructor::ResetCounts(); { |

