diff options
author | Xisco Fauli <anistenis@gmail.com> | 2016-05-29 20:46:29 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-05-30 07:22:37 +0000 |
commit | 7bc1c79c26e52d5196fb36eee5c2f12f12d49ba6 (patch) | |
tree | 1e02455e69bd90b6226ff055ef6e88c2da80b5bb | |
parent | ea01963baf5cc9f5a11f85e04fa61c56bc8a37f0 (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.hxx | 3 | ||||
-rw-r--r-- | unotools/source/config/defaultoptions.cxx | 4 |
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: */ |