loplugin:useuniqueptr in VCLObject

Change-Id: Ib31e3ea5c4ac59ff526e85b6156a06130ad20311
Reviewed-on: https://gerrit.libreoffice.org/58572
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 5 insertions, 5 deletions

diff --git a/include/canvas/vclwrapper.hxx b/include/canvas/vclwrapper.hxx
index e35cb5fa13b9..77cf8b705e11 100644
--- a/include/canvas/vclwrapper.hxx
+++ b/include/canvas/vclwrapper.hxx
@@ -63,7 +63,7 @@ namespace canvas
             // no explicit here. VCLObjects should be freely
             // constructible with Wrappees, and AFAIK there is no other
             // implicit conversion path that could cause harm here
-            VCLObject( Wrappee* pWrappee ) :
+            VCLObject( std::unique_ptr<Wrappee> pWrappee ) :
                 mpWrappee( pWrappee )
             {
             }
@@ -122,11 +122,11 @@ namespace canvas
                 // protecting object deletion with the solar mutex
                 SolarMutexGuard aGuard;
 
-                delete mpWrappee;
+                mpWrappee.reset();
             }
 
-            Wrappee*        operator->() { return mpWrappee; }
-            const Wrappee*  operator->() const { return mpWrappee; }
+            Wrappee*        operator->() { return mpWrappee.get(); }
+            const Wrappee*  operator->() const { return mpWrappee.get(); }
 
             Wrappee&        operator*() { return *mpWrappee; }
             const Wrappee&  operator*() const { return *mpWrappee; }
@@ -141,7 +141,7 @@ namespace canvas
 
         private:
 
-            Wrappee* mpWrappee;
+            std::unique_ptr<Wrappee> mpWrappee;
         };
 
     }