[Libreoffice-commits] core.git: include/svtools svtools/source
Xisco Fauli
anistenis at gmail.com
Thu May 26 07:45:43 UTC 2016
include/svtools/hyperlabel.hxx | 4 +---
svtools/source/control/hyperlabel.cxx | 2 +-
2 files changed, 2 insertions(+), 4 deletions(-)
New commits:
commit 61a70fb5094ca6edfef9500f358c517e6674cb0c
Author: Xisco Fauli <anistenis at gmail.com>
Date: Thu May 26 01:37:50 2016 +0200
tdf#89329: use unique_ptr for pImpl in hyperlabel
Change-Id: I697eb48591c53ba27c99fc86baa4b8b33e7c9fd1
Reviewed-on: https://gerrit.libreoffice.org/25474
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin at gmail.com>
diff --git a/include/svtools/hyperlabel.hxx b/include/svtools/hyperlabel.hxx
index b70e706..6e027c7 100644
--- a/include/svtools/hyperlabel.hxx
+++ b/include/svtools/hyperlabel.hxx
@@ -29,14 +29,12 @@
namespace svt
{
-
-
class HyperLabelImpl;
class HyperLabel : public FixedText
{
protected:
- HyperLabelImpl* m_pImpl;
+ std::unique_ptr<HyperLabelImpl> m_pImpl;
Link<HyperLabel*,void> maClickHdl;
virtual void MouseMove( const MouseEvent& rMEvt ) override;
diff --git a/svtools/source/control/hyperlabel.cxx b/svtools/source/control/hyperlabel.cxx
index 4530f99..9fb0879 100644
--- a/svtools/source/control/hyperlabel.cxx
+++ b/svtools/source/control/hyperlabel.cxx
@@ -159,7 +159,7 @@ namespace svt
void HyperLabel::dispose()
{
- delete m_pImpl;
+ m_pImpl.reset();
FixedText::dispose();
}
More information about the Libreoffice-commits
mailing list