From f18ea631bbe47a116a6b8ad6da5ab10a5be8158f Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Wed, 14 Oct 2015 15:17:31 +0200
Subject: Fix transfer_ptr not to rely on copy elision

---
 odb/details/transfer-ptr.hxx | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/odb/details/transfer-ptr.hxx b/odb/details/transfer-ptr.hxx
index 2e40e29..13f113e 100644
--- a/odb/details/transfer-ptr.hxx
+++ b/odb/details/transfer-ptr.hxx
@@ -30,9 +30,27 @@ namespace odb
 #ifdef ODB_CXX11_NULLPTR
       transfer_ptr (std::nullptr_t): p_ (0) {}
 #endif
-
       template <typename T1>
       transfer_ptr (std::unique_ptr<T1>&& p): p_ (p.release ()) {}
+
+    private:
+      transfer_ptr (const transfer_ptr&);
+      transfer_ptr& operator= (const transfer_ptr&);
+
+    public:
+      transfer_ptr (transfer_ptr&& p): p_ (p.transfer ()) {}
+#else
+    private:
+      transfer_ptr& operator= (const transfer_ptr&);
+
+    public:
+      // In our usage transfer_ptr is always created implicitly and
+      // never const. So while this is not very clean, it is legal.
+      // Plus it will all go away once we drop C++98 (I can hardly
+      // wait).
+      //
+      transfer_ptr (const transfer_ptr& p)
+          : p_ (const_cast<transfer_ptr&> (p).transfer ()) {}
 #endif
 
       ~transfer_ptr () {delete p_;}
-- 
cgit v1.1