[Libreoffice-commits] core.git: include/svtools svtools/source

Xisco Fauli anistenis at gmail.com
Thu May 26 07:44:30 UTC 2016


 include/svtools/roadmapwizard.hxx        |    4 +---
 svtools/source/dialogs/roadmapwizard.cxx |    3 +--
 2 files changed, 2 insertions(+), 5 deletions(-)

New commits:
commit 9a2bbaad2667c025744792948961606fbcc5029c
Author: Xisco Fauli <anistenis at gmail.com>
Date:   Thu May 26 01:40:23 2016 +0200

    tdf#89329: use unique_ptr for pImpl in roadmapwizard
    
    Change-Id: I37c24d830ae702579fe14ecba627d7f1d363eef6
    Reviewed-on: https://gerrit.libreoffice.org/25475
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noelgrandin at gmail.com>

diff --git a/include/svtools/roadmapwizard.hxx b/include/svtools/roadmapwizard.hxx
index 7e588e9..1fb57e3 100644
--- a/include/svtools/roadmapwizard.hxx
+++ b/include/svtools/roadmapwizard.hxx
@@ -26,8 +26,6 @@
 
 namespace svt
 {
-
-
     struct RoadmapWizardImpl;
     class RoadmapWizard;
 
@@ -64,7 +62,7 @@ namespace svt
     class SVT_DLLPUBLIC RoadmapWizard : public OWizardMachine, public RoadmapWizardTypes
     {
     private:
-        RoadmapWizardImpl*  m_pImpl;
+        std::unique_ptr<RoadmapWizardImpl>  m_pImpl;
 
     public:
         RoadmapWizard(
diff --git a/svtools/source/dialogs/roadmapwizard.cxx b/svtools/source/dialogs/roadmapwizard.cxx
index 1df26b6..4fa72e8 100644
--- a/svtools/source/dialogs/roadmapwizard.cxx
+++ b/svtools/source/dialogs/roadmapwizard.cxx
@@ -150,8 +150,7 @@ namespace svt
 
     void RoadmapWizard::dispose()
     {
-        delete m_pImpl;
-        m_pImpl = nullptr;
+        m_pImpl.reset();
         OWizardMachine::dispose();
     }
 


More information about the Libreoffice-commits mailing list