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

[Xisco Fauli]

 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 <aniste...@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 <c...@libreoffice.org>
    Reviewed-by: Noel Grandin <noelgran...@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();
     }
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits