less copying in SfxItemPropertyMap::getPropertyEntries

we can just expose the map now, and avoid copying all the properties

Change-Id: Icb22975508582268dfa96e41eb98ac01e7f51317
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113982
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 5 insertions, 8 deletions

diff --git a/linguistic/source/lngopt.cxx b/linguistic/source/lngopt.cxx
index a163b50dcad5..f7d7e4c0bf29 100644
--- a/linguistic/source/lngopt.cxx
+++ b/linguistic/source/lngopt.cxx
@@ -317,15 +317,12 @@ Sequence< PropertyValue > SAL_CALL
 {
     MutexGuard  aGuard( GetLinguMutex() );
 
-    PropertyEntryVector_t aPropEntries = aPropertyMap.getPropertyEntries();
     std::vector<PropertyValue> aProps;
-    aProps.reserve(aPropertyMap.getSize());
-
-    std::transform(aPropEntries.begin(), aPropEntries.end(), std::back_inserter(aProps),
-        [this](PropertyEntryVector_t::const_reference rPropEntry) {
-            return PropertyValue(rPropEntry.sName, rPropEntry.nWID,
-                                 aConfig.GetProperty(rPropEntry.nWID),
-                                 css::beans::PropertyState_DIRECT_VALUE); });
+    aProps.reserve(aPropertyMap.getPropertyEntries().size());
+    for(auto & rPair : aPropertyMap.getPropertyEntries())
+        aProps.push_back(PropertyValue(OUString(rPair.first), rPair.second.nWID,
+                                 aConfig.GetProperty(rPair.second.nWID),
+                                 css::beans::PropertyState_DIRECT_VALUE));
     return comphelper::containerToSequence(aProps);
 }