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

Bernhard Widl bernhard.widl at cib.de
Mon Mar 27 18:50:25 UTC 2017


 include/tubes/manager.hxx |    2 +-
 tubes/source/manager.cxx  |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 6d5fc2acf2354d72fb32e1cbae620d604c904cff
Author: Bernhard Widl <bernhard.widl at cib.de>
Date:   Mon Mar 27 18:40:08 2017 +0200

    tdf#89329: use unique_ptr for pImpl in tubes/manager
    
    Change-Id: Ic70883b58c9b3a0b56e6c07594a2758e04e7f5ee
    Reviewed-on: https://gerrit.libreoffice.org/35767
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Michael Stahl <mstahl at redhat.com>

diff --git a/include/tubes/manager.hxx b/include/tubes/manager.hxx
index 2ea14fce3888..063cf975e587 100644
--- a/include/tubes/manager.hxx
+++ b/include/tubes/manager.hxx
@@ -145,7 +145,7 @@ public:
     TUBES_DLLPUBLIC static void addSuffixToNames( const char* pName );
 
 private:
-    static TeleManagerImpl* pImpl;
+    static std::unique_ptr<TeleManagerImpl> pImpl;
 
     static ::osl::Mutex&    GetMutex();
 };
diff --git a/tubes/source/manager.cxx b/tubes/source/manager.cxx
index a742e2811491..08dacfb3ed58 100644
--- a/tubes/source/manager.cxx
+++ b/tubes/source/manager.cxx
@@ -436,7 +436,7 @@ bool TeleManager::init( bool bListen )
 
 void TeleManager::finalize()
 {
-    delete pImpl;
+    pImpl.reset();
 }
 
 bool TeleManager::createAccountManager()


More information about the Libreoffice-commits mailing list