mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tdf#89329: use unique_ptr for pImpl in configvaluecontainer

Browse Source 
Change-Id: Ide38cacc59a44515287d51dadb23836eaac14518
Reviewed-on: https://gerrit.libreoffice.org/25904
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
This commit is contained in:
Xisco Fauli 2016-06-04 23:09:16 +02:00 committed by Noel Grandin
parent c9ff3efd2c
commit a4a97fb56c
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
include/unotools/configvaluecontainer.hxx
View File

@ -23,6 +23,7 @@
#include <com/sun/star/uno/XComponentContext.hpp> #include <com/sun/star/uno/XComponentContext.hpp>
#include <osl/mutex.hxx> #include <osl/mutex.hxx>
#include <o3tl/typed_flags_set.hxx> #include <o3tl/typed_flags_set.hxx>
#include <memory>
enum class CVCFlags enum class CVCFlags
@ -65,8 +66,7 @@ namespace utl
class UNOTOOLS_DLLPUBLIC OConfigurationValueContainer class UNOTOOLS_DLLPUBLIC OConfigurationValueContainer
{ {
private: private:
OConfigurationValueContainerImpl* std::unique_ptr<OConfigurationValueContainerImpl> m_pImpl;
m_pImpl;
protected: protected:

1
unotools/source/config/configvaluecontainer.cxx
View File

@ -220,7 +220,6 @@ namespace utl
OConfigurationValueContainer::~OConfigurationValueContainer() OConfigurationValueContainer::~OConfigurationValueContainer()
{ {
delete m_pImpl;
} }
void OConfigurationValueContainer::implConstruct( const OUString& _rConfigLocation, void OConfigurationValueContainer::implConstruct( const OUString& _rConfigLocation,