[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