[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