diff options
author | Tor Egge <Tor.Egge@broadpark.no> | 2018-08-29 14:12:49 +0200 |
---|---|---|
committer | Tor Egge <Tor.Egge@broadpark.no> | 2018-08-29 14:20:13 +0200 |
commit | 860fb32d2a29fb8f69a43d913a3ea6694d036d73 (patch) | |
tree | 8fd458285121165566a42d2cb263f6639bd205ea /vespalib/src | |
parent | 4b4ee6503ad05926218b9a0d01a6ac0846bdf71c (diff) |
Add move assignment and move constructor to CloneablePtr and IdentifiablePtr.
This allows for use of std::unique_ptr rhs value instead of temporary
raw pointer.
Diffstat (limited to 'vespalib/src')
-rw-r--r-- | vespalib/src/vespa/vespalib/util/memory.h | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/vespalib/src/vespa/vespalib/util/memory.h b/vespalib/src/vespa/vespalib/util/memory.h index 7f51c4b553f..43be0531aec 100644 --- a/vespalib/src/vespa/vespalib/util/memory.h +++ b/vespalib/src/vespa/vespalib/util/memory.h @@ -349,6 +349,12 @@ public: } } + /** @brief move constructor, takes over ownership */ + CloneablePtr(std::unique_ptr<T> &&rhs) + : _p(rhs.release()) + { + } + /** @brief assignment operator, does deep copy using clone() */ CloneablePtr & operator = (const CloneablePtr & rhs) { if (this != &rhs) { @@ -358,6 +364,14 @@ public: return *this; } + /** @brief move assignment operator, takes over ownership */ + CloneablePtr &operator=(std::unique_ptr<T> &&rhs) + { + cleanup(); + _p = rhs.release(); + return *this; + } + /** @brief swap contents */ void swap(CloneablePtr & rhs) { std::swap(_p, rhs._p); } |