coverity#1371191 Missing move assignment operator

Change-Id: If543e0685f808cfe72eaa0ff60c87bb3b1a5ea42

1 files changed, 14 insertions, 2 deletions

diff --git a/include/canvas/vclwrapper.hxx b/include/canvas/vclwrapper.hxx
index dcef39c89a95..bdda0ce65f8f 100644
--- a/include/canvas/vclwrapper.hxx
+++ b/include/canvas/vclwrapper.hxx
@@ -79,6 +79,12 @@ namespace canvas
                     mpWrappee = nullptr;
             }
 
+            VCLObject( VCLObject&& rOrig )
+                : mpWrappee(rOrig.mpWrappee)
+            {
+                rOrig.mpWrappee = nullptr;
+            }
+
             // This object has value semantics, thus, forward copy
             // to wrappee
             VCLObject( const Wrappee& rOrig ) :
@@ -104,14 +110,20 @@ namespace canvas
                 return *this;
             }
 
+            VCLObject& operator=( VCLObject&& rhs )
+            {
+                std::swap(mpWrappee, rhs.mpWrappee);
+
+                return *this;
+            }
+
             ~VCLObject()
             {
                 // This here is the whole purpose of the template:
                 // protecting object deletion with the solar mutex
                 SolarMutexGuard aGuard;
 
-                if( mpWrappee )
-                    delete mpWrappee;
+                delete mpWrappee;
             }
 
             Wrappee*        operator->() { return mpWrappee; }