diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-17 15:59:54 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-25 07:23:31 +0100 |
| commit | aae1a702696f485a418fbf53500977b236976d38 (patch) | |
| tree | 7732ff1dfe7fbd24cbd9fb3ad4be4266949d06a9 /xmloff/source | |
| parent | eb8c84846bdba32c17b23e7da6742463085bec9d (diff) | |
loplugin:useuniqueptr in XMLEventImportHelper
Change-Id: I2369ac92fa2f30b98b5910563dd6d0a6f8ca6a0c
Reviewed-on: https://gerrit.libreoffice.org/48479
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'xmloff/source')
| -rw-r--r-- | xmloff/source/script/XMLEventImportHelper.cxx | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/xmloff/source/script/XMLEventImportHelper.cxx b/xmloff/source/script/XMLEventImportHelper.cxx index 84bcd8eb1d16..0e45525856af 100644 --- a/xmloff/source/script/XMLEventImportHelper.cxx +++ b/xmloff/source/script/XMLEventImportHelper.cxx @@ -50,7 +50,7 @@ XMLEventImportHelper::~XMLEventImportHelper() aFactoryMap.clear(); // delete name map - delete pEventNameMap; + pEventNameMap.reset(); } void XMLEventImportHelper::RegisterFactory( @@ -91,8 +91,8 @@ void XMLEventImportHelper::AddTranslationTable( void XMLEventImportHelper::PushTranslationTable() { // save old map and install new one - aEventNameMapVector.push_back(pEventNameMap); - pEventNameMap = new NameMap; + aEventNameMapVector.push_back(std::move(pEventNameMap)); + pEventNameMap.reset( new NameMap ); } void XMLEventImportHelper::PopTranslationTable() @@ -102,8 +102,7 @@ void XMLEventImportHelper::PopTranslationTable() if ( !aEventNameMapVector.empty() ) { // delete current and install old map - delete pEventNameMap; - pEventNameMap = aEventNameMapVector.back(); + pEventNameMap = std::move(aEventNameMapVector.back()); aEventNameMapVector.pop_back(); } } |