From a4a97fb56c526a0f3793e55826afce42f4b0eeef Mon Sep 17 00:00:00 2001 From: Xisco Fauli Date: Sat, 4 Jun 2016 23:09:16 +0200 Subject: [PATCH] tdf#89329: use unique_ptr for pImpl in configvaluecontainer Change-Id: Ide38cacc59a44515287d51dadb23836eaac14518 Reviewed-on: https://gerrit.libreoffice.org/25904 Tested-by: Jenkins Reviewed-by: Noel Grandin --- include/unotools/configvaluecontainer.hxx | 4 ++-- unotools/source/config/configvaluecontainer.cxx | 1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/include/unotools/configvaluecontainer.hxx b/include/unotools/configvaluecontainer.hxx index 970bc1861a0b..664f10aeef84 100644 --- a/include/unotools/configvaluecontainer.hxx +++ b/include/unotools/configvaluecontainer.hxx @@ -23,6 +23,7 @@ #include #include #include +#include enum class CVCFlags @@ -65,8 +66,7 @@ namespace utl class UNOTOOLS_DLLPUBLIC OConfigurationValueContainer { private: - OConfigurationValueContainerImpl* - m_pImpl; + std::unique_ptr m_pImpl; protected: diff --git a/unotools/source/config/configvaluecontainer.cxx b/unotools/source/config/configvaluecontainer.cxx index a13b52a80fbd..df0a002d4753 100644 --- a/unotools/source/config/configvaluecontainer.cxx +++ b/unotools/source/config/configvaluecontainer.cxx @@ -220,7 +220,6 @@ namespace utl OConfigurationValueContainer::~OConfigurationValueContainer() { - delete m_pImpl; } void OConfigurationValueContainer::implConstruct( const OUString& _rConfigLocation,