Use const_iterator and put them in for loop

Change-Id: I1dbfe5735580d940081298c372c12db8a3c3b3a3

comphelper/source/property/ChainablePropertySetInfo.cxx

@@ -84,9 +84,7 @@ Sequence< ::Property > SAL_CALL ChainablePropertySetInfo::getProperties()
         maProperties.realloc ( nSize );
         Property* pProperties = maProperties.getArray();
 
-        PropertyInfoHash::iterator aIter = maMap.begin();
-        const PropertyInfoHash::iterator aEnd = maMap.end();
-        for ( ; aIter != aEnd; ++aIter, ++pProperties)
+        for (PropertyInfoHash::const_iterator aIter(maMap.begin()), aEnd(maMap.end()); aIter != aEnd; ++aIter, ++pProperties)
         {
             PropertyInfo* pInfo = (*aIter).second;
 

comphelper/source/property/MasterPropertySetInfo.cxx

@@ -101,9 +101,7 @@ Sequence< ::Property > SAL_CALL MasterPropertySetInfo::getProperties()
         maProperties.realloc ( nSize );
         Property* pProperties = maProperties.getArray();
 
-        PropertyDataHash::iterator aIter = maMap.begin();
-        const PropertyDataHash::iterator aEnd = maMap.end();
-        for ( ; aIter != aEnd; ++aIter, ++pProperties)
+        for (PropertyDataHash::const_iterator aIter(maMap.begin()), aEnd(maMap.end()) ; aIter != aEnd; ++aIter, ++pProperties)
         {
             PropertyInfo* pInfo = (*aIter).second->mpInfo;