Fixed deleter move assignment and construction

Author: cschreib

include/oup/observable_unique_ptr.hpp

@@ -180,6 +180,17 @@ class observable_unique_ptr {
     explicit observable_unique_ptr(T* value, Deleter del) :
         observable_unique_ptr(allocate_block_(), value, std::move(del)) {}
 
+    /// Transfer ownership by implicit casting
+    /** \param value The pointer to take ownership from
+    *   \note After this observable_unique_ptr is created, the source
+    *         pointer is set to null and looses ownership.
+    */
+    observable_unique_ptr(observable_unique_ptr&& value) noexcept :
+        observable_unique_ptr(value.block, value.pointer, std::move(value.deleter)) {
+        value.block = nullptr;
+        value.pointer = nullptr;
+    }
+
     /// Transfer ownership by implicit casting
     /** \param value The pointer to take ownership from
     *   \note After this observable_unique_ptr is created, the source

tests/runtime_tests.cpp

@@ -71,7 +71,7 @@ TEST_CASE("owner move constructor with deleter", "[owner_construction]") {
             REQUIRE(instances == 1);
             REQUIRE(instances_deleter == 2);
             REQUIRE(ptr.get() != nullptr);
-            REQUIRE(ptr.get_deleter().state_ == 0);
+            REQUIRE(ptr.get_deleter().state_ == 42);
         }
 
         REQUIRE(instances == 0);
@@ -212,7 +212,7 @@ TEST_CASE("owner move assignment operator with deleter", "[owner_assignment]") {
             REQUIRE(instances == 1);
             REQUIRE(instances_deleter == 2);
             REQUIRE(ptr.get() != nullptr);
-            REQUIRE(ptr.get_deleter().state_ == 0);
+            REQUIRE(ptr.get_deleter().state_ == 42);
         }
 
         REQUIRE(instances == 0);