loplugin:sequenceloop in svtools..svx

Change-Id: I81b54f6db69491492b2bc16f48b2296ad96e137d
Reviewed-on: https://gerrit.libreoffice.org/77529
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

26 files changed, 46 insertions, 46 deletions

diff --git a/svx/source/accessibility/lookupcolorname.cxx b/svx/source/accessibility/lookupcolorname.cxx
index c33cb9b3be23..feffea69fe18 100644
--- a/svx/source/accessibility/lookupcolorname.cxx
+++ b/svx/source/accessibility/lookupcolorname.cxx
@@ -78,7 +78,7 @@ ColorNameMap::ColorNameMap() {
 
     // Fill the map to convert from numerical color values to names.
     if (xNA.is())
-        for (const auto& rName : aNames)
+        for (const auto& rName : std::as_const(aNames))
         {
             // Get the numerical value for the i-th color name.
             try
diff --git a/svx/source/core/graphichelper.cxx b/svx/source/core/graphichelper.cxx
index e9fa4c819a24..7302d752d072 100644
--- a/svx/source/core/graphichelper.cxx
+++ b/svx/source/core/graphichelper.cxx
@@ -149,7 +149,7 @@ bool lcl_ExecuteFilterDialog( const Sequence< PropertyValue >& rPropsForDialog,
             if( xFilterDialog->execute() )
             {
                 bStatus = true;
-                Sequence< PropertyValue > aPropsFromDialog = xFilterProperties->getPropertyValues();
+                const Sequence< PropertyValue > aPropsFromDialog = xFilterProperties->getPropertyValues();
                 for ( const auto& rProp : aPropsFromDialog )
                 {
                     if (rProp.Name == "FilterData")
@@ -297,7 +297,7 @@ OUString GraphicHelper::ExportGraphic(weld::Window* pParent, const Graphic& rGra
                     sal_Int32 nWidth = 0;
                     sal_Int32 nHeight = 0;
 
-                    for (const auto& rProp : aFilterData)
+                    for (const auto& rProp : std::as_const(aFilterData))
                     {
                         if (rProp.Name == "PixelWidth")
                         {
diff --git a/svx/source/customshapes/EnhancedCustomShape2d.cxx b/svx/source/customshapes/EnhancedCustomShape2d.cxx
index cb226c3c9333..c5b296941ebb 100644
--- a/svx/source/customshapes/EnhancedCustomShape2d.cxx
+++ b/svx/source/customshapes/EnhancedCustomShape2d.cxx
@@ -2040,7 +2040,7 @@ void EnhancedCustomShape2d::CreateSubPath(
     sal_Int32 nSegInfoSize = seqSegments.getLength();
     if ( !nSegInfoSize )
     {
-        for ( const EnhancedCustomShapeParameterPair& rCoordinate : seqCoordinates )
+        for ( const EnhancedCustomShapeParameterPair& rCoordinate : std::as_const(seqCoordinates) )
         {
             const Point aTempPoint(GetPoint( rCoordinate, true, true ));
             aNewB2DPolygon.append(basegfx::B2DPoint(aTempPoint.X(), aTempPoint.Y()));
@@ -2996,7 +2996,7 @@ void EnhancedCustomShape2d::ApplyGluePoints( SdrObject* pObj )
 {
     if ( pObj )
     {
-        for ( const auto& rGluePoint : seqGluePoints )
+        for ( const auto& rGluePoint : std::as_const(seqGluePoints) )
         {
             SdrGluePoint aGluePoint;
 
diff --git a/svx/source/customshapes/EnhancedCustomShapeEngine.cxx b/svx/source/customshapes/EnhancedCustomShapeEngine.cxx
index 7e7bcca224a8..95cb473a86a2 100644
--- a/svx/source/customshapes/EnhancedCustomShapeEngine.cxx
+++ b/svx/source/customshapes/EnhancedCustomShapeEngine.cxx
@@ -123,7 +123,7 @@ void SAL_CALL EnhancedCustomShapeEngine::initialize( const Sequence< Any >& aArg
         if ( rArgument >>= aParameter )
             break;
     }
-    for ( const beans::PropertyValue& rProp : aParameter )
+    for ( const beans::PropertyValue& rProp : std::as_const(aParameter) )
     {
         if ( rProp.Name == "CustomShape" )
             rProp.Value >>= mxShape;
diff --git a/svx/source/dialog/ClassificationCommon.cxx b/svx/source/dialog/ClassificationCommon.cxx
index c8e37076f531..7c609b4d70fe 100644
--- a/svx/source/dialog/ClassificationCommon.cxx
+++ b/svx/source/dialog/ClassificationCommon.cxx
@@ -68,7 +68,7 @@ bool containsProperty(uno::Sequence<beans::Property> const& rProperties, OUStrin
 void removeAllProperties(uno::Reference<beans::XPropertyContainer> const& rxPropertyContainer)
 {
     uno::Reference<beans::XPropertySet> xPropertySet(rxPropertyContainer, uno::UNO_QUERY);
-    uno::Sequence<beans::Property> aProperties
+    const uno::Sequence<beans::Property> aProperties
         = xPropertySet->getPropertySetInfo()->getProperties();
 
     for (const beans::Property& rProperty : aProperties)
diff --git a/svx/source/fmcomp/gridcell.cxx b/svx/source/fmcomp/gridcell.cxx
index ab5705fd4cab..db802b41a2d6 100644
--- a/svx/source/fmcomp/gridcell.cxx
+++ b/svx/source/fmcomp/gridcell.cxx
@@ -2424,7 +2424,7 @@ void DbComboBox::SetList(const Any& rItems)
     css::uno::Sequence<OUString> aTest;
     if (rItems >>= aTest)
     {
-        for (const OUString& rString : aTest)
+        for (const OUString& rString : std::as_const(aTest))
              pField->InsertEntry(rString);
 
         // tell the grid control that this controller is invalid and has to be re-initialized
@@ -2545,7 +2545,7 @@ void DbListBox::SetList(const Any& rItems)
     {
         if (aTest.hasElements())
         {
-            for (const OUString& rString : aTest)
+            for (const OUString& rString : std::as_const(aTest))
                  pField->InsertEntry(rString);
 
             m_rColumn.getModel()->getPropertyValue(FM_PROP_VALUE_SEQ) >>= m_aValueList;
@@ -2707,13 +2707,13 @@ void DbFilterField::SetList(const Any& rItems, bool bComboBox)
         if (bComboBox)
         {
             ComboBox* pField = static_cast<ComboBox*>(m_pWindow.get());
-            for (const OUString& rString : aTest)
+            for (const OUString& rString : std::as_const(aTest))
                 pField->InsertEntry(rString);
         }
         else
         {
             ListBox* pField = static_cast<ListBox*>(m_pWindow.get());
-            for (const OUString& rString : aTest)
+            for (const OUString& rString : std::as_const(aTest))
                 pField->InsertEntry(rString);
 
             m_rColumn.getModel()->getPropertyValue(FM_PROP_VALUE_SEQ) >>= m_aValueList;
diff --git a/svx/source/form/dataaccessdescriptor.cxx b/svx/source/form/dataaccessdescriptor.cxx
index 361be0c11ce7..acad7b4a78f6 100644
--- a/svx/source/form/dataaccessdescriptor.cxx
+++ b/svx/source/form/dataaccessdescriptor.cxx
@@ -133,7 +133,7 @@ namespace svx
         }
 
         // build a PropertyValue sequence with the current values
-        Sequence< Property > aProperties = xPropInfo->getProperties();
+        const Sequence< Property > aProperties = xPropInfo->getProperties();
 
         Sequence< PropertyValue > aValues(aProperties.getLength());
         PropertyValue* pValues = aValues.getArray();
diff --git a/svx/source/form/databaselocationinput.cxx b/svx/source/form/databaselocationinput.cxx
index 96093e8134ca..620ab6af29d9 100644
--- a/svx/source/form/databaselocationinput.cxx
+++ b/svx/source/form/databaselocationinput.cxx
@@ -94,7 +94,7 @@ namespace svx
 
         // forward the allowed extensions to the input control
         OUStringBuffer aExtensionList;
-        for ( auto const & extension : m_aFilterExtensions )
+        for ( auto const & extension : std::as_const(m_aFilterExtensions) )
         {
             aExtensionList.append( extension );
             aExtensionList.append( ';' );
diff --git a/svx/source/form/datanavi.cxx b/svx/source/form/datanavi.cxx
index 8390adda415c..263e405937c5 100644
--- a/svx/source/form/datanavi.cxx
+++ b/svx/source/form/datanavi.cxx
@@ -1901,7 +1901,7 @@ namespace svxform
                     if ( xContainer.is() )
                     {
                         m_xDataContainer = xContainer;
-                        Sequence< OUString > aNameList = m_xDataContainer->getElementNames();
+                        const Sequence< OUString > aNameList = m_xDataContainer->getElementNames();
                         for ( const OUString& rName : aNameList )
                         {
                             Any aAny = m_xDataContainer->getByName( rName );
@@ -2405,7 +2405,7 @@ namespace svxform
         try
         {
             // get property names & infos, and iterate over target properties
-            Sequence< Property > aProperties = xTo->getPropertySetInfo()->getProperties();
+            const Sequence< Property > aProperties = xTo->getPropertySetInfo()->getProperties();
             Reference< XPropertySetInfo > xFromInfo = xFrom->getPropertySetInfo();
             for ( const Property& rProperty : aProperties )
             {
@@ -2669,7 +2669,7 @@ namespace svxform
                         xModel->getDataTypeRepository();
                     if ( xDataTypes.is() )
                     {
-                        Sequence< OUString > aNameList = xDataTypes->getElementNames();
+                        const Sequence< OUString > aNameList = xDataTypes->getElementNames();
                         for ( const OUString& rName : aNameList )
                             m_xDataTypeLB->append_text(rName);
                     }
@@ -2958,7 +2958,7 @@ namespace svxform
         try
         {
             int nRow = 0;
-            Sequence< OUString > aAllNames = m_rNamespaces->getElementNames();
+            const Sequence< OUString > aAllNames = m_rNamespaces->getElementNames();
             for ( const OUString& sPrefix : aAllNames )
             {
                 if ( m_rNamespaces->hasByName( sPrefix ) )
diff --git a/svx/source/form/filtnav.cxx b/svx/source/form/filtnav.cxx
index e68594ffd4af..1ea13ebe31c0 100644
--- a/svx/source/form/filtnav.cxx
+++ b/svx/source/form/filtnav.cxx
@@ -556,7 +556,7 @@ void FmFilterModel::Update(const Reference< XIndexAccess > & xControllers, FmPar
             // insert the existing filters for the form
             OUString aTitle(SvxResId(RID_STR_FILTER_FILTER_FOR));
 
-            Sequence< Sequence< OUString > > aExpressions = xFilterController->getPredicateExpressions();
+            const Sequence< Sequence< OUString > > aExpressions = xFilterController->getPredicateExpressions();
             for ( auto const & conjunctionTerm : aExpressions )
             {
                 // we always display one row, even if there's no term to be displayed
diff --git a/svx/source/form/fmexch.cxx b/svx/source/form/fmexch.cxx
index 139aeb4ffeb6..ce1087a26eab 100644
--- a/svx/source/form/fmexch.cxx
+++ b/svx/source/form/fmexch.cxx
@@ -250,7 +250,7 @@ namespace svxform
         ListBoxEntrySet aEmpty;
         m_aSelectedEntries.swap( aEmpty );
 
-        for (const css::uno::Sequence<sal_uInt32>& rPaths : m_aControlPaths)
+        for (const css::uno::Sequence<sal_uInt32>& rPaths : std::as_const(m_aControlPaths))
         {
             SvTreeListEntry* pSearch = pRoot;
             for (const sal_uInt32 nThisPath : rPaths)
diff --git a/svx/source/form/fmshimp.cxx b/svx/source/form/fmshimp.cxx
index 553c03cbeb18..2d42276f187e 100644
--- a/svx/source/form/fmshimp.cxx
+++ b/svx/source/form/fmshimp.cxx
@@ -402,7 +402,7 @@ namespace
         for (const ScriptEventDescriptor& rCurrent : rTransferIfAvailable)
         {
             // search the model/control idl classes for the event described by pCurrent
-            for (Sequence< Type>* pCurrentArray : { &aModelListeners, &aControlListeners })
+            for (const Sequence< Type>* pCurrentArray : { &aModelListeners, &aControlListeners })
             {
                 for (const Type& rCurrentListener : *pCurrentArray)
                 {
@@ -2823,7 +2823,7 @@ Reference< XControl> FmXFormShell::impl_getControl_Lock(const Reference<XControl
     {
         Reference< XControlContainer> xControlContainer(getControlContainerForView_Lock(), UNO_SET_THROW);
 
-        Sequence< Reference< XControl > > seqControls( xControlContainer->getControls() );
+        const Sequence< Reference< XControl > > seqControls( xControlContainer->getControls() );
         // ... that I can then search
         for (Reference< XControl > const & control : seqControls)
         {
@@ -3342,7 +3342,7 @@ void FmXFormShell::CreateExternalView_Lock()
                 aColumnProps.realloc(nExistentDescs + aProps.getLength());
                 pColumnProps = aColumnProps.getArray() + nExistentDescs;
 
-                for (const Property& rProp : aProps)
+                for (const Property& rProp : std::as_const(aProps))
                 {
                     if (rProp.Name == FM_PROP_LABEL)
                         // already set
diff --git a/svx/source/form/formcontroller.cxx b/svx/source/form/formcontroller.cxx
index 9043ae73ac64..39d2f8f6a81d 100644
--- a/svx/source/form/formcontroller.cxx
+++ b/svx/source/form/formcontroller.cxx
@@ -1985,7 +1985,7 @@ void FormController::setContainer(const Reference< XControlContainer > & xContai
         m_aFilterComponents.clear();
 
         // collecting the controls
-        for ( const Reference< XControl >& rControl : m_aControls )
+        for ( const Reference< XControl >& rControl : std::as_const(m_aControls) )
             implControlRemoved( rControl, true );
 
         // make database-specific things
@@ -2001,7 +2001,7 @@ void FormController::setContainer(const Reference< XControlContainer > & xContai
     // What controls belong to the container?
     if (xContainer.is() && xTabModel.is())
     {
-        Sequence< Reference< XControlModel > > aModels = xTabModel->getControlModels();
+        const Sequence< Reference< XControlModel > > aModels = xTabModel->getControlModels();
         Sequence< Reference< XControl > > aAllControls = xContainer->getControls();
 
         sal_Int32 nCount = aModels.getLength();
@@ -2066,7 +2066,7 @@ Sequence< Reference< XControl > > FormController::getControls()
         if (!xModel.is())
             return m_aControls;
 
-        Sequence< Reference< XControlModel > > aControlModels = xModel->getControlModels();
+        const Sequence< Reference< XControlModel > > aControlModels = xModel->getControlModels();
         sal_Int32 nModels = aControlModels.getLength();
 
         Sequence< Reference< XControl > > aNewControls(nModels);
@@ -2175,7 +2175,7 @@ void FormController::setLocks()
 {
     OSL_ENSURE( !impl_isDisposed_nofail(), "FormController: already disposed!" );
     // lock/unlock all controls connected to a data source
-    for ( const Reference< XControl >& rControl : m_aControls )
+    for ( const Reference< XControl >& rControl : std::as_const(m_aControls) )
         setControlLock( rControl );
 }
 
@@ -2313,7 +2313,7 @@ void FormController::startListening()
     m_bModified  = false;
 
     // now register at bound fields
-    for ( const Reference< XControl >& rControl : m_aControls )
+    for ( const Reference< XControl >& rControl : std::as_const(m_aControls) )
         startControlModifyListening( rControl );
 }
 
@@ -2324,7 +2324,7 @@ void FormController::stopListening()
     m_bModified  = false;
 
     // now register at bound fields
-    for ( const Reference< XControl >& rControl : m_aControls )
+    for ( const Reference< XControl >& rControl : std::as_const(m_aControls) )
         stopControlModifyListening( rControl );
 }
 
@@ -2635,7 +2635,7 @@ void FormController::unload()
 
 void FormController::removeBoundFieldListener()
 {
-    for ( const Reference< XControl >& rControl : m_aControls )
+    for ( const Reference< XControl >& rControl : std::as_const(m_aControls) )
     {
         Reference< XPropertySet > xProp( rControl, UNO_QUERY );
         if ( xProp.is() )
@@ -3005,7 +3005,7 @@ void FormController::setFilter(::std::vector<FmFieldInfo>& rFieldInfos)
 
     if (m_xComposer.is())
     {
-        Sequence< Sequence < PropertyValue > > aFilterRows = m_xComposer->getStructuredFilter();
+        const Sequence< Sequence < PropertyValue > > aFilterRows = m_xComposer->getStructuredFilter();
 
         // ok, we receive the list of filters as sequence of fieldnames, value
         // now we have to transform the fieldname into UI names, that could be a label of the field or
@@ -3526,7 +3526,7 @@ Reference< XControl > FormController::locateControl( const Reference< XControlMo
 {
     try
     {
-        Sequence< Reference< XControl > > aControls( getControls() );
+        const Sequence< Reference< XControl > > aControls( getControls() );
 
         for ( auto const & control : aControls )
         {
diff --git a/svx/source/form/navigatortree.cxx b/svx/source/form/navigatortree.cxx
index 2e2fa562dd67..b1cee44678bf 100644
--- a/svx/source/form/navigatortree.cxx
+++ b/svx/source/form/navigatortree.cxx
@@ -966,7 +966,7 @@ namespace svxform
                     // should only contain hidden controls
 #endif // (OSL_DEBUG_LEVEL > 0)
                 Reference< XPropertySetInfo >  xPropInfo( xCurrent->getPropertySetInfo());
-                Sequence< Property> seqAllCurrentProps = xPropInfo->getProperties();
+                const Sequence< Property> seqAllCurrentProps = xPropInfo->getProperties();
                 for (Property const & currentProp : seqAllCurrentProps)
                 {
                     if (((currentProp.Attributes & PropertyAttribute::READONLY) == 0) && (currentProp.Name != FM_PROP_NAME))
diff --git a/svx/source/form/tabwin.cxx b/svx/source/form/tabwin.cxx
index fd8fe32c4628..dd8296696964 100644
--- a/svx/source/form/tabwin.cxx
+++ b/svx/source/form/tabwin.cxx
@@ -89,7 +89,7 @@ struct ColumnInfo
 
 static void lcl_addToList( SvTreeListBox& _rListBox, const uno::Reference< container::XNameAccess>& i_xColumns )
 {
-    uno::Sequence< OUString > aEntries = i_xColumns->getElementNames();
+    const uno::Sequence< OUString > aEntries = i_xColumns->getElementNames();
     for ( const OUString& rEntry : aEntries )
     {
         uno::Reference< beans::XPropertySet> xColumn(i_xColumns->getByName(rEntry),UNO_QUERY_THROW);
diff --git a/svx/source/gallery2/galbrws2.cxx b/svx/source/gallery2/galbrws2.cxx
index 92734278eb85..88e95b540f93 100644
--- a/svx/source/gallery2/galbrws2.cxx
+++ b/svx/source/gallery2/galbrws2.cxx
@@ -192,7 +192,7 @@ void SAL_CALL GalleryThemePopup::statusChanged(
             else if ( ( rEvent.State >>= sItems ) && sItems.hasElements() )
             {
                 sal_uInt16 nId = 1;
-                for ( const OUString& rStr : sItems )
+                for ( const OUString& rStr : std::as_const(sItems) )
                 {
                     mpBackgroundPopup->InsertItem( nId, rStr );
                     nId++;
diff --git a/svx/source/items/galleryitem.cxx b/svx/source/items/galleryitem.cxx
index 56d1de655c34..07e88d07adcf 100644
--- a/svx/source/items/galleryitem.cxx
+++ b/svx/source/items/galleryitem.cxx
@@ -80,7 +80,7 @@ bool SvxGalleryItem::PutValue( const css::uno::Any& rVal, sal_uInt8 /* nMemberId
     css::uno::Reference< css::lang::XComponent > xDrawing;
     css::uno::Reference< css::graphic::XGraphic > xGraphic;
 
-    for ( const css::beans::PropertyValue& rProp : aSeq )
+    for ( const css::beans::PropertyValue& rProp : std::as_const(aSeq) )
     {
         if ( rProp.Name == SVXGALLERYITEM_TYPE )
         {
diff --git a/svx/source/items/viewlayoutitem.cxx b/svx/source/items/viewlayoutitem.cxx
index 0d4f714f6104..1d31d37f029d 100644
--- a/svx/source/items/viewlayoutitem.cxx
+++ b/svx/source/items/viewlayoutitem.cxx
@@ -113,7 +113,7 @@ bool SvxViewLayoutItem::PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId
                 bool  bBookMode = false;
                 bool bAllConverted( true );
                 sal_Int16 nConvertedCount( 0 );
-                for ( const auto& rProp : aSeq )
+                for ( const auto& rProp : std::as_const(aSeq) )
                 {
                     if ( rProp.Name == VIEWLAYOUT_PARAM_COLUMNS )
                     {
diff --git a/svx/source/items/zoomslideritem.cxx b/svx/source/items/zoomslideritem.cxx
index faeeb67a7bb4..ad53388b22a5 100644
--- a/svx/source/items/zoomslideritem.cxx
+++ b/svx/source/items/zoomslideritem.cxx
@@ -121,7 +121,7 @@ bool SvxZoomSliderItem::PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId
                     sal_Int16 nConvertedCount( 0 );
                     sal_Int32 nMinZoom( 0 ), nMaxZoom( 0 );
 
-                    for ( const auto& rProp : aSeq )
+                    for ( const auto& rProp : std::as_const(aSeq) )
                     {
                         if ( rProp.Name == ZOOMSLIDER_PARAM_CURRENTZOOM )
                         {
diff --git a/svx/source/mnuctrls/smarttagmenu.cxx b/svx/source/mnuctrls/smarttagmenu.cxx
index 106234bde46c..dd55876b85ed 100644
--- a/svx/source/mnuctrls/smarttagmenu.cxx
+++ b/svx/source/mnuctrls/smarttagmenu.cxx
@@ -76,7 +76,7 @@ void SmartTagMenuController::statusChanged( const css::frame::FeatureStateEvent&
     OUString aApplicationName;
     OUString aRangeText;
 
-    for ( const auto& aProperty : aProperties )
+    for ( const auto& aProperty : std::as_const(aProperties) )
     {
         if ( aProperty.Name == "ActionComponents" )
             aProperty.Value >>= aActionComponents;
diff --git a/svx/source/smarttags/SmartTagMgr.cxx b/svx/source/smarttags/SmartTagMgr.cxx
index be65e6d0fff7..640fb8a7938c 100644
--- a/svx/source/smarttags/SmartTagMgr.cxx
+++ b/svx/source/smarttags/SmartTagMgr.cxx
@@ -431,7 +431,7 @@ void SmartTagMgr::ReadConfiguration( bool bExcludedTypes, bool bRecognize )
             Sequence< OUString > aValues;
             aAny >>= aValues;
 
-            for ( const auto& rValue : aValues )
+            for ( const auto& rValue : std::as_const(aValues) )
                 maDisabledSmartTagTypes.insert( rValue );
         }
 
diff --git a/svx/source/unodialogs/textconversiondlgs/chinese_dictionarydialog.cxx b/svx/source/unodialogs/textconversiondlgs/chinese_dictionarydialog.cxx
index 84eb7b700548..b5cb64c7af02 100644
--- a/svx/source/unodialogs/textconversiondlgs/chinese_dictionarydialog.cxx
+++ b/svx/source/unodialogs/textconversiondlgs/chinese_dictionarydialog.cxx
@@ -120,7 +120,7 @@ void DictionaryList::refillFromDictionary( sal_Int32 nTextConversionOptions )
     if(!m_xDictionary.is())
         return;
 
-    Sequence< OUString > aLeftList(  m_xDictionary->getConversionEntries( linguistic2::ConversionDirection_FROM_LEFT ) );
+    const Sequence< OUString > aLeftList(  m_xDictionary->getConversionEntries( linguistic2::ConversionDirection_FROM_LEFT ) );
 
     Reference< linguistic2::XConversionPropertyType > xPropertyType( m_xDictionary, uno::UNO_QUERY );
 
diff --git a/svx/source/unogallery/unogalthemeprovider.cxx b/svx/source/unogallery/unogalthemeprovider.cxx
index a0b72e997435..58c4234ef899 100644
--- a/svx/source/unogallery/unogalthemeprovider.cxx
+++ b/svx/source/unogallery/unogalthemeprovider.cxx
@@ -129,7 +129,7 @@ void SAL_CALL GalleryThemeProvider::initialize( const uno::Sequence< uno::Any >&
             break;
     }
 
-    for( const beans::PropertyValue& rProp : aParams )
+    for( const beans::PropertyValue& rProp : std::as_const(aParams) )
     {
         if ( rProp.Name == "ProvideHiddenThemes" )
             rProp.Value >>= mbHiddenThemes;
diff --git a/svx/source/xml/xmlxtexp.cxx b/svx/source/xml/xmlxtexp.cxx
index 8175f3f7589e..400146ce579b 100644
--- a/svx/source/xml/xmlxtexp.cxx
+++ b/svx/source/xml/xmlxtexp.cxx
@@ -372,7 +372,7 @@ bool SvxXMLXTableExportComponent::exportTable() throw()
 
             SvXMLElementExport aElem( *this, XML_NAMESPACE_OOO, pEleName, true, true );
 
-            Sequence< OUString > aNames = mxTable->getElementNames();
+            const Sequence< OUString > aNames = mxTable->getElementNames();
             Any aAny;
 
             for( const OUString& rName : aNames )
diff --git a/svx/source/xoutdev/xattr.cxx b/svx/source/xoutdev/xattr.cxx
index 9c4727447c3f..af477725dc23 100644
--- a/svx/source/xoutdev/xattr.cxx
+++ b/svx/source/xoutdev/xattr.cxx
@@ -742,7 +742,7 @@ bool XLineDashItem::PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId )
                 css::drawing::LineDash aLineDash;
                 OUString aName;
                 bool bLineDash( false );
-                for ( const auto& rProp : aPropSeq )
+                for ( const auto& rProp : std::as_const(aPropSeq) )
                 {
                     if ( rProp.Name == "Name" )
                         rProp.Value >>= aName;
@@ -2105,7 +2105,7 @@ bool XFillGradientItem::PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId
                 css::awt::Gradient aGradient2;
                 OUString aName;
                 bool bGradient( false );
-                for ( const auto& rProp : aPropSeq )
+                for ( const auto& rProp : std::as_const(aPropSeq) )
                 {
                     if ( rProp.Name == "Name" )
                         rProp.Value >>= aName;
@@ -2498,7 +2498,7 @@ bool XFillHatchItem::PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId )
                 css::drawing::Hatch aUnoHatch;
                 OUString aName;
                 bool bHatch( false );
-                for ( const auto& rProp : aPropSeq )
+                for ( const auto& rProp : std::as_const(aPropSeq) )
                 {
                     if ( rProp.Name == "Name" )
                         rProp.Value >>= aName;
diff --git a/svx/source/xoutdev/xattrbmp.cxx b/svx/source/xoutdev/xattrbmp.cxx
index bb284e903e36..448005de3879 100644
--- a/svx/source/xoutdev/xattrbmp.cxx
+++ b/svx/source/xoutdev/xattrbmp.cxx
@@ -265,7 +265,7 @@ bool XFillBitmapItem::PutValue( const css::uno::Any& rVal, sal_uInt8 nMemberId )
         uno::Sequence< beans::PropertyValue >   aPropSeq;
         if( rVal >>= aPropSeq )
         {
-            for ( const auto& rProp : aPropSeq )
+            for ( const auto& rProp : std::as_const(aPropSeq) )
             {
                 if ( rProp.Name == "Name" )
                     bSetName = (rProp.Value >>= aName);