[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