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

Xisco Fauli anistenis at gmail.com
Thu May 26 07:43:16 UTC 2016


 include/svtools/svtabbx.hxx       |    2 +-
 svtools/source/contnr/svtabbx.cxx |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit bb694dfef762327a5446338aa1a1ad23ed8970ed
Author: Xisco Fauli <anistenis at gmail.com>
Date:   Thu May 26 01:44:07 2016 +0200

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

diff --git a/include/svtools/svtabbx.hxx b/include/svtools/svtabbx.hxx
index bf86082..6a0b866 100644
--- a/include/svtools/svtabbx.hxx
+++ b/include/svtools/svtabbx.hxx
@@ -128,7 +128,7 @@ private:
     typedef ::std::vector< css::uno::Reference< css::accessibility::XAccessible > > AccessibleChildren;
 
     bool                            m_bFirstPaint;
-    ::svt::SvHeaderTabListBoxImpl*  m_pImpl;
+    std::unique_ptr<::svt::SvHeaderTabListBoxImpl>  m_pImpl;
     ::svt::IAccessibleTabListBox*   m_pAccessible;
     AccessibleChildren              m_aAccessibleChildren;
 
diff --git a/svtools/source/contnr/svtabbx.cxx b/svtools/source/contnr/svtabbx.cxx
index f3a9d92..f0345af 100644
--- a/svtools/source/contnr/svtabbx.cxx
+++ b/svtools/source/contnr/svtabbx.cxx
@@ -519,7 +519,7 @@ SvHeaderTabListBox::~SvHeaderTabListBox()
 
 void SvHeaderTabListBox::dispose()
 {
-    delete m_pImpl;
+    m_pImpl.reset();
     SvTabListBox::dispose();
 }
 


More information about the Libreoffice-commits mailing list