[Libreoffice-commits] core.git: include/comphelper

Bjoern Michaelsen bjoern.michaelsen at canonical.com
Tue Feb 24 09:13:17 PST 2015


 include/comphelper/propertysequence.hxx |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 18a0d519440b42cd90c628730b2e404b4203e8eb
Author: Bjoern Michaelsen <bjoern.michaelsen at canonical.com>
Date:   Tue Feb 24 17:25:49 2015 +0100

    use by-ref iteration, omit pointless std::move() and add missing inline
    
    Change-Id: I45081f5f80c4a571af0abebd23a4fed6623498a0

diff --git a/include/comphelper/propertysequence.hxx b/include/comphelper/propertysequence.hxx
index ce28e4f..7cd9494 100644
--- a/include/comphelper/propertysequence.hxx
+++ b/include/comphelper/propertysequence.hxx
@@ -18,18 +18,18 @@
 
 namespace comphelper
 {
-    ::com::sun::star::uno::Sequence< ::com::sun::star::beans::PropertyValue > InitPropertySequence(
+    inline ::com::sun::star::uno::Sequence< ::com::sun::star::beans::PropertyValue > InitPropertySequence(
         ::std::initializer_list< ::std::pair< OUString, ::com::sun::star::uno::Any > > vInit)
     {
         ::com::sun::star::uno::Sequence< ::com::sun::star::beans::PropertyValue> vResult{static_cast<sal_Int32>(vInit.size())};
         size_t nCount{0};
-        for(auto aEntry : vInit)
+        for(const auto& aEntry : vInit)
         {
-            vResult[nCount].Name = std::move(aEntry.first);
-            vResult[nCount].Value = std::move(aEntry.second);
+            vResult[nCount].Name = aEntry.first;
+            vResult[nCount].Value = aEntry.second;
             ++nCount;
         }
-        return std::move(vResult);
+        return vResult;
     }
 }   // namespace comphelper
 


More information about the Libreoffice-commits mailing list