summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorXisco Fauli <anistenis@gmail.com>2016-05-29 20:46:29 +0200
committerNoel Grandin <noelgrandin@gmail.com>2016-05-30 07:22:37 +0000
commit7bc1c79c26e52d5196fb36eee5c2f12f12d49ba6 (patch)
tree1e02455e69bd90b6226ff055ef6e88c2da80b5bb
parentea01963baf5cc9f5a11f85e04fa61c56bc8a37f0 (diff)
tdf#89329: use unique_ptr for pImpl in defaultoptions
Change-Id: I355888c44d1413f8d0ebd7a247649b79647c38ec Reviewed-on: https://gerrit.libreoffice.org/25629 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
-rw-r--r--include/unotools/defaultoptions.hxx3
-rw-r--r--unotools/source/config/defaultoptions.cxx4
2 files changed, 4 insertions, 3 deletions
diff --git a/include/unotools/defaultoptions.hxx b/include/unotools/defaultoptions.hxx
index 7877087e6a94..f4937662d4c2 100644
--- a/include/unotools/defaultoptions.hxx
+++ b/include/unotools/defaultoptions.hxx
@@ -22,6 +22,7 @@
#include <unotools/unotoolsdllapi.h>
#include <rtl/ustring.hxx>
#include <unotools/options.hxx>
+#include <memory>
class SvtDefaultOptions_Impl;
@@ -29,7 +30,7 @@ class SvtDefaultOptions_Impl;
class SAL_WARN_UNUSED UNOTOOLS_DLLPUBLIC SvtDefaultOptions : public utl::detail::Options
{
private:
- SvtDefaultOptions_Impl* pImp;
+ std::unique_ptr<SvtDefaultOptions_Impl> pImpl;
public:
diff --git a/unotools/source/config/defaultoptions.cxx b/unotools/source/config/defaultoptions.cxx
index 92d5a046c686..6f021d4240c4 100644
--- a/unotools/source/config/defaultoptions.cxx
+++ b/unotools/source/config/defaultoptions.cxx
@@ -326,7 +326,7 @@ SvtDefaultOptions::SvtDefaultOptions()
ItemHolder1::holdConfigItem(E_DEFAULTOPTIONS);
}
++nRefCount;
- pImp = pOptions;
+ pImpl.reset(pOptions);
}
SvtDefaultOptions::~SvtDefaultOptions()
@@ -343,7 +343,7 @@ SvtDefaultOptions::~SvtDefaultOptions()
OUString SvtDefaultOptions::GetDefaultPath( sal_uInt16 nId ) const
{
- return pImp->GetDefaultPath( nId );
+ return pImpl->GetDefaultPath( nId );
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */