diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-09-30 15:10:05 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-10-01 08:26:43 +0200 |
commit | 9b3643820bf821c8be728ed20a8cb7ed05cabef2 (patch) | |
tree | 85b3cf0347f5e887b3b6f36aad33f6bb805452d0 /framework/source/accelerators | |
parent | 4f2465f599020b356dc37f3132049a103dd32486 (diff) |
loplugin:stringadd in extensions..framework
Change-Id: I0a0788cdbde292cae582982b0000ee6d5a2056d7
Reviewed-on: https://gerrit.libreoffice.org/79888
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'framework/source/accelerators')
-rw-r--r-- | framework/source/accelerators/presethandler.cxx | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/framework/source/accelerators/presethandler.cxx b/framework/source/accelerators/presethandler.cxx index b81abece09c1..3b4aa1704ed7 100644 --- a/framework/source/accelerators/presethandler.cxx +++ b/framework/source/accelerators/presethandler.cxx @@ -479,11 +479,8 @@ void PresetHandler::copyPresetToTarget(const OUString& sPreset, return; } - OUString sPresetFile(sPreset); - sPresetFile += ".xml"; - - OUString sTargetFile(sTarget); - sTargetFile += ".xml"; + OUString sPresetFile = sPreset + ".xml"; + OUString sTargetFile = sTarget + ".xml"; // remove existing elements before you try to copy the preset to that location ... // Otherwise w will get an ElementExistException inside copyElementTo()! @@ -510,8 +507,7 @@ css::uno::Reference< css::io::XStream > PresetHandler::openPreset(const OUString if (!xFolder.is()) return css::uno::Reference< css::io::XStream >(); - OUString sFile(sPreset); - sFile += ".xml"; + OUString sFile = sPreset + ".xml"; // inform user about errors (use original exceptions!) css::uno::Reference< css::io::XStream > xStream = xFolder->openStreamElement(sFile, css::embed::ElementModes::READ); |