[Libreoffice-commits] core.git: oox/source

Julien Nabet serval2412 at yahoo.fr
Sat Mar 31 06:17:29 UTC 2018


 oox/source/core/relations.cxx                      |   14 +-
 oox/source/core/xmlfilterbase.cxx                  |    4 
 oox/source/docprop/docprophandler.cxx              |    8 -
 oox/source/drawingml/chart/axisconverter.cxx       |    5 
 oox/source/drawingml/chart/chartconverter.cxx      |    4 
 oox/source/drawingml/chart/converterbase.cxx       |    4 
 oox/source/drawingml/chart/datasourceconverter.cxx |    5 
 oox/source/drawingml/chart/plotareaconverter.cxx   |   51 ++++---
 oox/source/drawingml/chart/seriesconverter.cxx     |   20 +--
 oox/source/drawingml/chart/typegroupconverter.cxx  |   20 +--
 oox/source/drawingml/color.cxx                     |   64 ++++-----
 oox/source/drawingml/customshapeproperties.cxx     |   15 +-
 oox/source/drawingml/diagram/diagram.cxx           |  136 ++++++++-------------
 oox/source/drawingml/effectproperties.cxx          |    6 
 oox/source/drawingml/fillproperties.cxx            |   13 --
 oox/source/drawingml/shape.cxx                     |   14 --
 oox/source/drawingml/table/tableproperties.cxx     |   22 +--
 17 files changed, 192 insertions(+), 213 deletions(-)

New commits:
commit 22b061267efe28f3611fd38534a59e8d74bf1e77
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Fri Mar 30 22:14:59 2018 +0200

    Use for-range loops in oox (part1)
    
    Change-Id: I3b8b3cdc818509598bc766f7eed27454c74c4e2d
    Reviewed-on: https://gerrit.libreoffice.org/52175
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>

diff --git a/oox/source/core/relations.cxx b/oox/source/core/relations.cxx
index 8bb5448d9bb8..49a84d368e56 100644
--- a/oox/source/core/relations.cxx
+++ b/oox/source/core/relations.cxx
@@ -64,19 +64,19 @@ const Relation* Relations::getRelationFromRelId( const OUString& rId ) const
 
 const Relation* Relations::getRelationFromFirstType( const OUString& rType ) const
 {
-    for( ::std::map< OUString, Relation >::const_iterator aIt = maMap.begin(), aEnd = maMap.end(); aIt != aEnd; ++aIt )
-        if( aIt->second.maType.equalsIgnoreAsciiCase( rType ) )
-            return &aIt->second;
+    for (auto const& elem : maMap)
+        if( elem.second.maType.equalsIgnoreAsciiCase( rType ) )
+            return &elem.second;
     return nullptr;
 }
 
 RelationsRef Relations::getRelationsFromTypeFromOfficeDoc( const OUString& rType ) const
 {
     RelationsRef xRelations( new Relations( maFragmentPath ) );
-    for( ::std::map< OUString, Relation >::const_iterator aIt = maMap.begin(), aEnd = maMap.end(); aIt != aEnd; ++aIt )
-        if( aIt->second.maType.equalsIgnoreAsciiCase( createOfficeDocRelationTypeTransitional(rType) ) ||
-                aIt->second.maType.equalsIgnoreAsciiCase( createOfficeDocRelationTypeStrict(rType) ))
-            xRelations->maMap[ aIt->first ] = aIt->second;
+    for (auto const& elem : maMap)
+        if( elem.second.maType.equalsIgnoreAsciiCase( createOfficeDocRelationTypeTransitional(rType) ) ||
+                elem.second.maType.equalsIgnoreAsciiCase( createOfficeDocRelationTypeStrict(rType) ))
+            xRelations->maMap[ elem.first ] = elem.second;
     return xRelations;
 }
 
diff --git a/oox/source/core/xmlfilterbase.cxx b/oox/source/core/xmlfilterbase.cxx
index 58333fb6c229..2bbb23a38721 100644
--- a/oox/source/core/xmlfilterbase.cxx
+++ b/oox/source/core/xmlfilterbase.cxx
@@ -164,8 +164,8 @@ void registerNamespaces( FastParser& rParser )
     for (sal_Int32 i = 0; i < ids.getLength(); ++i)
         aSet.insert(ids[i].Second);
 
-    for (std::set<sal_Int32>::iterator it = aSet.begin(); it != aSet.end(); ++it)
-        rParser.registerNamespace(*it);
+    for (auto const& elem : aSet)
+        rParser.registerNamespace(elem);
 }
 
 } // namespace
diff --git a/oox/source/docprop/docprophandler.cxx b/oox/source/docprop/docprophandler.cxx
index 206455c66307..ae8d4829a91c 100644
--- a/oox/source/docprop/docprophandler.cxx
+++ b/oox/source/docprop/docprophandler.cxx
@@ -201,9 +201,11 @@ uno::Sequence< OUString > OOXMLDocPropHandler::GetKeywordsSet( const OUString& a
         {
             uno::Sequence< OUString > aResult( aUtf8Result.size() );
             OUString* pResultValues = aResult.getArray();
-            for ( std::vector< std::string >::const_iterator i = aUtf8Result.begin();
-                  i != aUtf8Result.end(); ++i, ++pResultValues )
-                *pResultValues = OUString( i->c_str(), static_cast< sal_Int32 >( i->size() ),RTL_TEXTENCODING_UTF8 );
+            for (auto const& elem : aUtf8Result)
+            {
+                *pResultValues = OUString( elem.c_str(), static_cast< sal_Int32 >( elem.size() ),RTL_TEXTENCODING_UTF8 );
+                ++pResultValues;
+            }
 
             return aResult;
         }
diff --git a/oox/source/drawingml/chart/axisconverter.cxx b/oox/source/drawingml/chart/axisconverter.cxx
index 727c0f599489..5aead418e983 100644
--- a/oox/source/drawingml/chart/axisconverter.cxx
+++ b/oox/source/drawingml/chart/axisconverter.cxx
@@ -116,10 +116,9 @@ bool isPercent( const RefVector<TypeGroupConverter>& rTypeGroups )
     if (rTypeGroups.empty())
         return false;
 
-    RefVector<TypeGroupConverter>::const_iterator it = rTypeGroups.begin(), itEnd = rTypeGroups.end();
-    for (; it != itEnd; ++it)
+    for (auto const& typeGroup : rTypeGroups)
     {
-        TypeGroupConverter& rConv = **it;
+        TypeGroupConverter& rConv = *typeGroup;
         if (!rConv.isPercent())
             return false;
     }
diff --git a/oox/source/drawingml/chart/chartconverter.cxx b/oox/source/drawingml/chart/chartconverter.cxx
index 34ef0e72d822..4992758396ac 100644
--- a/oox/source/drawingml/chart/chartconverter.cxx
+++ b/oox/source/drawingml/chart/chartconverter.cxx
@@ -134,8 +134,8 @@ Reference< XDataSequence > ChartConverter::createDataSequence(
         {
             // create a single-row array from constant source data
             Matrix< Any > aMatrix( rDataSeq.mnPointCount, 1 );
-            for( DataSequenceModel::AnyMap::const_iterator aDIt = rDataSeq.maData.begin(), aDEnd = rDataSeq.maData.end(); aDIt != aDEnd; ++aDIt )
-                *aMatrix.at(aDIt->first, 0) = aDIt->second;
+            for (auto const& elem : rDataSeq.maData)
+                *aMatrix.at(elem.first, 0) = elem.second;
 
             aRangeRep = lclGenerateApiArray( aMatrix );
         }
diff --git a/oox/source/drawingml/chart/converterbase.cxx b/oox/source/drawingml/chart/converterbase.cxx
index 35ed591936f5..a0535671dbc6 100644
--- a/oox/source/drawingml/chart/converterbase.cxx
+++ b/oox/source/drawingml/chart/converterbase.cxx
@@ -274,8 +274,8 @@ void ConverterRoot::convertTitlePositions()
     try
     {
         Reference< cssc::XChartDocument > xChart1Doc( mxData->mxDoc, UNO_QUERY_THROW );
-        for( ConverterData::TitleMap::iterator aIt = mxData->maTitles.begin(), aEnd = mxData->maTitles.end(); aIt != aEnd; ++aIt )
-            aIt->second.convertTitlePos( *this, xChart1Doc );
+        for (auto & title : mxData->maTitles)
+            title.second.convertTitlePos( *this, xChart1Doc );
     }
     catch( Exception& )
     {
diff --git a/oox/source/drawingml/chart/datasourceconverter.cxx b/oox/source/drawingml/chart/datasourceconverter.cxx
index 701525d0bde0..bcfbf1bf8406 100644
--- a/oox/source/drawingml/chart/datasourceconverter.cxx
+++ b/oox/source/drawingml/chart/datasourceconverter.cxx
@@ -52,10 +52,9 @@ Reference< XDataSequence > DataSequenceConverter::createDataSequence( const OUSt
         mrModel.mnPointCount = std::min<sal_Int32>(mrModel.mnPointCount, 1);
         OUStringBuffer aTitle;
         bool bFirst = true;
-        for(DataSequenceModel::AnyMap::const_iterator itr = mrModel.maData.begin(),
-                itrEnd = mrModel.maData.end(); itr != itrEnd; ++itr)
+        for (auto const& elem : mrModel.maData)
         {
-            Any aAny = itr->second;
+            Any aAny = elem.second;
             if(aAny.has<OUString>())
             {
                 if(!bFirst)
diff --git a/oox/source/drawingml/chart/plotareaconverter.cxx b/oox/source/drawingml/chart/plotareaconverter.cxx
index 5861b8ef537c..f89044d277de 100644
--- a/oox/source/drawingml/chart/plotareaconverter.cxx
+++ b/oox/source/drawingml/chart/plotareaconverter.cxx
@@ -111,8 +111,8 @@ void AxesSetConverter::convertFromModel( const Reference< XDiagram >& rxDiagram,
     // create type group converter objects for all type groups
     typedef RefVector< TypeGroupConverter > TypeGroupConvVector;
     TypeGroupConvVector aTypeGroups;
-    for( AxesSetModel::TypeGroupVector::iterator aIt = mrModel.maTypeGroups.begin(), aEnd = mrModel.maTypeGroups.end(); aIt != aEnd; ++aIt )
-        aTypeGroups.push_back( std::make_shared<TypeGroupConverter>( *this, **aIt ) );
+    for (auto const& typeGroup : mrModel.maTypeGroups)
+        aTypeGroups.push_back( std::make_shared<TypeGroupConverter>( *this, *typeGroup ) );
 
     OSL_ENSURE( !aTypeGroups.empty(), "AxesSetConverter::convertFromModel - no type groups in axes set" );
     if( !aTypeGroups.empty() ) try
@@ -175,8 +175,8 @@ void AxesSetConverter::convertFromModel( const Reference< XDiagram >& rxDiagram,
             }
 
             // convert all chart type groups, this converts all series data and formatting
-            for( TypeGroupConvVector::iterator aTIt = aTypeGroups.begin(), aTEnd = aTypeGroups.end(); aTIt != aTEnd; ++aTIt )
-                (*aTIt)->convertFromModel( rxDiagram, xCoordSystem, nAxesSetIdx, bSupportsVaryColorsByPoint );
+            for (auto const& typeGroup : aTypeGroups)
+                typeGroup->convertFromModel( rxDiagram, xCoordSystem, nAxesSetIdx, bSupportsVaryColorsByPoint );
         }
     }
     catch( Exception& )
@@ -335,36 +335,40 @@ void PlotAreaConverter::convertFromModel( View3DModel& rView3DModel )
     // store all axis models in a map, keyed by axis identifier
     typedef ModelMap< sal_Int32, AxisModel > AxisMap;
     AxisMap aAxisMap;
-    for( PlotAreaModel::AxisVector::iterator aAIt = mrModel.maAxes.begin(), aAEnd = mrModel.maAxes.end(); aAIt != aAEnd; ++aAIt )
+    for (auto const& axis : mrModel.maAxes)
     {
-        PlotAreaModel::AxisVector::value_type xAxis = *aAIt;
-        OSL_ENSURE( xAxis->mnAxisId >= 0, "PlotAreaConverter::convertFromModel - invalid axis identifier" );
-        OSL_ENSURE( !aAxisMap.has( xAxis->mnAxisId ), "PlotAreaConverter::convertFromModel - axis identifiers not unique" );
-        if( xAxis->mnAxisId != -1 )
-            aAxisMap[ xAxis->mnAxisId ] = xAxis;
+        OSL_ENSURE( axis->mnAxisId >= 0, "PlotAreaConverter::convertFromModel - invalid axis identifier" );
+        OSL_ENSURE( !aAxisMap.has( axis->mnAxisId ), "PlotAreaConverter::convertFromModel - axis identifiers not unique" );
+        if( axis->mnAxisId != -1 )
+            aAxisMap[ axis->mnAxisId ] = axis;
     }
 
     // group the type group models into different axes sets
     typedef ModelVector< AxesSetModel > AxesSetVector;
     AxesSetVector aAxesSets;
     sal_Int32 nMaxSeriesIdx = -1;
-    for( PlotAreaModel::TypeGroupVector::iterator aTIt = mrModel.maTypeGroups.begin(), aTEnd = mrModel.maTypeGroups.end(); aTIt != aTEnd; ++aTIt )
+    for (auto const& typeGroup : mrModel.maTypeGroups)
     {
-        PlotAreaModel::TypeGroupVector::value_type xTypeGroup = *aTIt;
-        if( !xTypeGroup->maSeries.empty() )
+        if( !typeGroup->maSeries.empty() )
         {
             // try to find a compatible axes set for the type group
             AxesSetModel* pAxesSet = nullptr;
-            for( AxesSetVector::iterator aASIt = aAxesSets.begin(), aASEnd = aAxesSets.end(); !pAxesSet && (aASIt != aASEnd); ++aASIt )
-                if( (*aASIt)->maTypeGroups.front()->maAxisIds == xTypeGroup->maAxisIds )
-                    pAxesSet = aASIt->get();
+            for (auto const& axesSet : aAxesSets)
+            {
+                if( axesSet->maTypeGroups.front()->maAxisIds == typeGroup->maAxisIds )
+                {
+                    pAxesSet = axesSet.get();
+                    if (pAxesSet)
+                        break;
+                }
+            }
 
             // not possible to insert into an existing axes set -> start a new axes set
             if( !pAxesSet )
             {
                 pAxesSet = &aAxesSets.create();
                 // find axis models used by the type group
-                const TypeGroupModel::AxisIdVector& rAxisIds = xTypeGroup->maAxisIds;
+                const TypeGroupModel::AxisIdVector& rAxisIds = typeGroup->maAxisIds;
                 if( rAxisIds.size() >= 1 )
                     pAxesSet->maAxes[ API_X_AXIS ] = aAxisMap.get( rAxisIds[ 0 ] );
                 if( rAxisIds.size() >= 2 )
@@ -374,11 +378,11 @@ void PlotAreaConverter::convertFromModel( View3DModel& rView3DModel )
             }
 
             // insert the type group model
-            pAxesSet->maTypeGroups.push_back( xTypeGroup );
+            pAxesSet->maTypeGroups.push_back( typeGroup );
 
             // collect the maximum series index for automatic series formatting
-            for( TypeGroupModel::SeriesVector::iterator aSIt = xTypeGroup->maSeries.begin(), aSEnd = xTypeGroup->maSeries.end(); aSIt != aSEnd; ++aSIt )
-                nMaxSeriesIdx = ::std::max( nMaxSeriesIdx, (*aSIt)->mnIndex );
+            for (auto const& elemSeries : typeGroup->maSeries)
+                nMaxSeriesIdx = ::std::max( nMaxSeriesIdx, elemSeries->mnIndex );
         }
     }
     getFormatter().setMaxSeriesIndex( nMaxSeriesIdx );
@@ -387,10 +391,10 @@ void PlotAreaConverter::convertFromModel( View3DModel& rView3DModel )
     bool bSupportsVaryColorsByPoint = mrModel.maTypeGroups.size() == 1;
 
     // convert all axes sets
-    for( AxesSetVector::iterator aASBeg = aAxesSets.begin(), aASIt = aASBeg, aASEnd = aAxesSets.end(); aASIt != aASEnd; ++aASIt )
+    sal_Int32 nAxesSetIdx = 0;
+    for (auto const& axesSet : aAxesSets)
     {
-        AxesSetConverter aAxesSetConv( *this, **aASIt );
-        sal_Int32 nAxesSetIdx = static_cast< sal_Int32 >( aASIt - aASBeg );
+        AxesSetConverter aAxesSetConv(*this, *axesSet);
         aAxesSetConv.convertFromModel( xDiagram, rView3DModel, nAxesSetIdx, bSupportsVaryColorsByPoint );
         if( nAxesSetIdx == 0 )
         {
@@ -403,6 +407,7 @@ void PlotAreaConverter::convertFromModel( View3DModel& rView3DModel )
         {
             maAutoTitle.clear();
         }
+        ++nAxesSetIdx;
     }
 
     DataTableConverter dataTableConverter (*this, mrModel.mxDataTable.getOrCreate());
diff --git a/oox/source/drawingml/chart/seriesconverter.cxx b/oox/source/drawingml/chart/seriesconverter.cxx
index ac5977d115da..7e35fc6f2628 100644
--- a/oox/source/drawingml/chart/seriesconverter.cxx
+++ b/oox/source/drawingml/chart/seriesconverter.cxx
@@ -368,12 +368,12 @@ void DataLabelsConverter::convertFromModel( const Reference< XDataSeries >& rxDa
     }
 
     // data point label settings
-    for( DataLabelsModel::DataLabelVector::iterator aIt = mrModel.maPointLabels.begin(), aEnd = mrModel.maPointLabels.end(); aIt != aEnd; ++aIt )
+    for (auto const& pointLabel : mrModel.maPointLabels)
     {
-        if ((*aIt)->maNumberFormat.maFormatCode.isEmpty())
-            (*aIt)->maNumberFormat = mrModel.maNumberFormat;
+        if (pointLabel->maNumberFormat.maFormatCode.isEmpty())
+            pointLabel->maNumberFormat = mrModel.maNumberFormat;
 
-        DataLabelConverter aLabelConv( *this, **aIt );
+        DataLabelConverter aLabelConv(*this, *pointLabel);
         aLabelConv.convertFromModel( rxDataSeries, rTypeGroup, aPropSet );
     }
 }
@@ -712,16 +712,16 @@ Reference< XDataSeries > SeriesConverter::createDataSeries( const TypeGroupConve
     }
 
     // error bars
-    for( SeriesModel::ErrorBarVector::iterator aIt = mrModel.maErrorBars.begin(), aEnd = mrModel.maErrorBars.end(); aIt != aEnd; ++aIt )
+    for (auto const& errorBar : mrModel.maErrorBars)
     {
-        ErrorBarConverter aErrorBarConv( *this, **aIt );
+        ErrorBarConverter aErrorBarConv(*this, *errorBar);
         aErrorBarConv.convertFromModel( xDataSeries );
     }
 
     // trendlines
-    for( SeriesModel::TrendlineVector::iterator aIt = mrModel.maTrendlines.begin(), aEnd = mrModel.maTrendlines.end(); aIt != aEnd; ++aIt )
+    for (auto const& trendLine : mrModel.maTrendlines)
     {
-        TrendlineConverter aTrendlineConv( *this, **aIt );
+        TrendlineConverter aTrendlineConv(*this, *trendLine);
         aTrendlineConv.convertFromModel( xDataSeries );
     }
 
@@ -775,9 +775,9 @@ Reference< XDataSeries > SeriesConverter::createDataSeries( const TypeGroupConve
     }
 
     // data point settings
-    for( SeriesModel::DataPointVector::iterator aIt = mrModel.maPoints.begin(), aEnd = mrModel.maPoints.end(); aIt != aEnd; ++aIt )
+    for (auto const& point : mrModel.maPoints)
     {
-        DataPointConverter aPointConv( *this, **aIt );
+        DataPointConverter aPointConv(*this, *point);
         aPointConv.convertFromModel( xDataSeries, rTypeGroup, mrModel );
     }
 
diff --git a/oox/source/drawingml/chart/typegroupconverter.cxx b/oox/source/drawingml/chart/typegroupconverter.cxx
index 66de768d8d8a..fb8a50f0203b 100644
--- a/oox/source/drawingml/chart/typegroupconverter.cxx
+++ b/oox/source/drawingml/chart/typegroupconverter.cxx
@@ -270,19 +270,21 @@ Reference< XLabeledDataSequence > TypeGroupConverter::createCategorySequence()
     /*  Find first existing category sequence. The bahaviour of Excel 2007 is
         different to Excel 2003, which always used the category sequence of the
         first series, even if it was empty. */
-    for( TypeGroupModel::SeriesVector::iterator aIt = mrModel.maSeries.begin(), aEnd = mrModel.maSeries.end(); !xLabeledSeq.is() && (aIt != aEnd); ++aIt )
+    for (auto const& elem : mrModel.maSeries)
     {
-        if( (*aIt)->maSources.has( SeriesModel::CATEGORIES ) )
+        if( elem->maSources.has( SeriesModel::CATEGORIES ) )
         {
-            SeriesConverter aSeriesConv( *this, **aIt );
+            SeriesConverter aSeriesConv(*this, *elem);
             xLabeledSeq = aSeriesConv.createCategorySequence( "categories" );
         }
-        else if( nMaxValues <= 0 && (*aIt)->maSources.has( SeriesModel::VALUES ) )
+        else if( nMaxValues <= 0 && elem->maSources.has( SeriesModel::VALUES ) )
         {
-            DataSourceModel *pValues = (*aIt)->maSources.get( SeriesModel::VALUES ).get();
+            DataSourceModel *pValues = elem->maSources.get( SeriesModel::VALUES ).get();
             if( pValues->mxDataSeq.is() )
                 nMaxValues = pValues->mxDataSeq.get()->maData.size();
         }
+        if (xLabeledSeq.is())
+            break;
     }
     /* n#839727 Create Category Sequence when none are found */
     if( !xLabeledSeq.is() && mrModel.maSeries.size() > 0 ) {
@@ -340,8 +342,8 @@ void TypeGroupConverter::convertFromModel( const Reference< XDiagram >& rxDiagra
         // create converter objects for all series models
         typedef RefVector< SeriesConverter > SeriesConvVector;
         SeriesConvVector aSeries;
-        for( TypeGroupModel::SeriesVector::iterator aIt = mrModel.maSeries.begin(), aEnd = mrModel.maSeries.end(); aIt != aEnd; ++aIt )
-            aSeries.push_back( std::make_shared<SeriesConverter>( *this, **aIt ) );
+        for (auto const& elemSeries : mrModel.maSeries)
+            aSeries.push_back( std::make_shared<SeriesConverter>(*this, *elemSeries) );
 
         // reverse series order for some unstacked 2D chart types
         if( maTypeInfo.mbReverseSeries && !mb3dChart && !isStacked() && !isPercent() )
@@ -413,9 +415,9 @@ void TypeGroupConverter::convertFromModel( const Reference< XDiagram >& rxDiagra
         }
         else
         {
-            for( SeriesConvVector::iterator aIt = aSeries.begin(), aEnd = aSeries.end(); aIt != aEnd; ++aIt )
+            for (auto const& elem : aSeries)
             {
-                SeriesConverter& rSeriesConv = **aIt;
+                SeriesConverter& rSeriesConv = *elem;
                 Reference< XDataSeries > xDataSeries = rSeriesConv.createDataSeries( *this, bVaryColorsByPoint );
                 insertDataSeries( xChartType, xDataSeries, nAxesSetIdx );
 
diff --git a/oox/source/drawingml/color.cxx b/oox/source/drawingml/color.cxx
index ec4a18008719..f0ca6f509404 100644
--- a/oox/source/drawingml/color.cxx
+++ b/oox/source/drawingml/color.cxx
@@ -461,42 +461,42 @@ void Color::clearTransparence()
     // if color is UNUSED or turns to UNUSED in setResolvedRgb, do not perform transformations
     if( meMode != COLOR_UNUSED )
     {
-        for( TransformVec::const_iterator aIt = maTransforms.begin(), aEnd = maTransforms.end(); aIt != aEnd; ++aIt )
+        for (auto const& transform : maTransforms)
         {
-            switch( aIt->mnToken )
+            switch( transform.mnToken )
             {
-                case XML_red:       toCrgb(); lclSetValue( mnC1, aIt->mnValue );    break;
-                case XML_redMod:    toCrgb(); lclModValue( mnC1, aIt->mnValue );    break;
-                case XML_redOff:    toCrgb(); lclOffValue( mnC1, aIt->mnValue );    break;
-                case XML_green:     toCrgb(); lclSetValue( mnC2, aIt->mnValue );    break;
-                case XML_greenMod:  toCrgb(); lclModValue( mnC2, aIt->mnValue );    break;
-                case XML_greenOff:  toCrgb(); lclOffValue( mnC2, aIt->mnValue );    break;
-                case XML_blue:      toCrgb(); lclSetValue( mnC3, aIt->mnValue );    break;
-                case XML_blueMod:   toCrgb(); lclModValue( mnC3, aIt->mnValue );    break;
-                case XML_blueOff:   toCrgb(); lclOffValue( mnC3, aIt->mnValue );    break;
-
-                case XML_hue:       toHsl(); lclSetValue( mnC1, aIt->mnValue, MAX_DEGREE ); break;
-                case XML_hueMod:    toHsl(); lclModValue( mnC1, aIt->mnValue, MAX_DEGREE ); break;
-                case XML_hueOff:    toHsl(); lclOffValue( mnC1, aIt->mnValue, MAX_DEGREE ); break;
-                case XML_sat:       toHsl(); lclSetValue( mnC2, aIt->mnValue );             break;
-                case XML_satMod:    toHsl(); lclModValue( mnC2, aIt->mnValue );             break;
-                case XML_satOff:    toHsl(); lclOffValue( mnC2, aIt->mnValue );             break;
+                case XML_red:       toCrgb(); lclSetValue( mnC1, transform.mnValue );    break;
+                case XML_redMod:    toCrgb(); lclModValue( mnC1, transform.mnValue );    break;
+                case XML_redOff:    toCrgb(); lclOffValue( mnC1, transform.mnValue );    break;
+                case XML_green:     toCrgb(); lclSetValue( mnC2, transform.mnValue );    break;
+                case XML_greenMod:  toCrgb(); lclModValue( mnC2, transform.mnValue );    break;
+                case XML_greenOff:  toCrgb(); lclOffValue( mnC2, transform.mnValue );    break;
+                case XML_blue:      toCrgb(); lclSetValue( mnC3, transform.mnValue );    break;
+                case XML_blueMod:   toCrgb(); lclModValue( mnC3, transform.mnValue );    break;
+                case XML_blueOff:   toCrgb(); lclOffValue( mnC3, transform.mnValue );    break;
+
+                case XML_hue:       toHsl(); lclSetValue( mnC1, transform.mnValue, MAX_DEGREE ); break;
+                case XML_hueMod:    toHsl(); lclModValue( mnC1, transform.mnValue, MAX_DEGREE ); break;
+                case XML_hueOff:    toHsl(); lclOffValue( mnC1, transform.mnValue, MAX_DEGREE ); break;
+                case XML_sat:       toHsl(); lclSetValue( mnC2, transform.mnValue );             break;
+                case XML_satMod:    toHsl(); lclModValue( mnC2, transform.mnValue );             break;
+                case XML_satOff:    toHsl(); lclOffValue( mnC2, transform.mnValue );             break;
 
                 case XML_lum:
                     toHsl();
-                    lclSetValue( mnC3, aIt->mnValue );
+                    lclSetValue( mnC3, transform.mnValue );
                     // if color changes to black or white, it will stay gray if luminance changes again
                     if( (mnC3 == 0) || (mnC3 == MAX_PERCENT) ) mnC2 = 0;
                 break;
                 case XML_lumMod:
                     toHsl();
-                    lclModValue( mnC3, aIt->mnValue );
+                    lclModValue( mnC3, transform.mnValue );
                     // if color changes to black or white, it will stay gray if luminance changes again
                     if( (mnC3 == 0) || (mnC3 == MAX_PERCENT) ) mnC2 = 0;
                 break;
                 case XML_lumOff:
                     toHsl();
-                    lclOffValue( mnC3, aIt->mnValue );
+                    lclOffValue( mnC3, transform.mnValue );
                     // if color changes to black or white, it will stay gray if luminance changes again
                     if( (mnC3 == 0) || (mnC3 == MAX_PERCENT) ) mnC2 = 0;
                 break;
@@ -504,10 +504,10 @@ void Color::clearTransparence()
                 case XML_shade:
                     // shade: 0% = black, 100% = original color
                     toCrgb();
-                    OSL_ENSURE( (0 <= aIt->mnValue) && (aIt->mnValue <= MAX_PERCENT), "Color::getColor - invalid shade value" );
-                    if( (0 <= aIt->mnValue) && (aIt->mnValue <= MAX_PERCENT) )
+                    OSL_ENSURE( (0 <= transform.mnValue) && (transform.mnValue <= MAX_PERCENT), "Color::getColor - invalid shade value" );
+                    if( (0 <= transform.mnValue) && (transform.mnValue <= MAX_PERCENT) )
                     {
-                        double fFactor = static_cast< double >( aIt->mnValue ) / MAX_PERCENT;
+                        double fFactor = static_cast< double >( transform.mnValue ) / MAX_PERCENT;
                         mnC1 = static_cast< sal_Int32 >( mnC1 * fFactor );
                         mnC2 = static_cast< sal_Int32 >( mnC2 * fFactor );
                         mnC3 = static_cast< sal_Int32 >( mnC3 * fFactor );
@@ -516,10 +516,10 @@ void Color::clearTransparence()
                 case XML_tint:
                     // tint: 0% = white, 100% = original color
                     toCrgb();
-                    OSL_ENSURE( (0 <= aIt->mnValue) && (aIt->mnValue <= MAX_PERCENT), "Color::getColor - invalid tint value" );
-                    if( (0 <= aIt->mnValue) && (aIt->mnValue <= MAX_PERCENT) )
+                    OSL_ENSURE( (0 <= transform.mnValue) && (transform.mnValue <= MAX_PERCENT), "Color::getColor - invalid tint value" );
+                    if( (0 <= transform.mnValue) && (transform.mnValue <= MAX_PERCENT) )
                     {
-                        double fFactor = static_cast< double >( aIt->mnValue ) / MAX_PERCENT;
+                        double fFactor = static_cast< double >( transform.mnValue ) / MAX_PERCENT;
                         mnC1 = static_cast< sal_Int32 >( MAX_PERCENT - (MAX_PERCENT - mnC1) * fFactor );
                         mnC2 = static_cast< sal_Int32 >( MAX_PERCENT - (MAX_PERCENT - mnC2) * fFactor );
                         mnC3 = static_cast< sal_Int32 >( MAX_PERCENT - (MAX_PERCENT - mnC3) * fFactor );
@@ -528,17 +528,17 @@ void Color::clearTransparence()
                 case XLS_TOKEN( tint ):
                     // Excel tint: move luminance relative to current value
                     toHsl();
-                    OSL_ENSURE( (-MAX_PERCENT <= aIt->mnValue) && (aIt->mnValue <= MAX_PERCENT), "Color::getColor - invalid tint value" );
-                    if( (-MAX_PERCENT <= aIt->mnValue) && (aIt->mnValue < 0) )
+                    OSL_ENSURE( (-MAX_PERCENT <= transform.mnValue) && (transform.mnValue <= MAX_PERCENT), "Color::getColor - invalid tint value" );
+                    if( (-MAX_PERCENT <= transform.mnValue) && (transform.mnValue < 0) )
                     {
                         // negative: luminance towards 0% (black)
-                        lclModValue( mnC3, aIt->mnValue + MAX_PERCENT );
+                        lclModValue( mnC3, transform.mnValue + MAX_PERCENT );
                     }
-                    else if( (0 < aIt->mnValue) && (aIt->mnValue <= MAX_PERCENT) )
+                    else if( (0 < transform.mnValue) && (transform.mnValue <= MAX_PERCENT) )
                     {
                         // positive: luminance towards 100% (white)
                         mnC3 = MAX_PERCENT - mnC3;
-                        lclModValue( mnC3, MAX_PERCENT - aIt->mnValue );
+                        lclModValue( mnC3, MAX_PERCENT - transform.mnValue );
                         mnC3 = MAX_PERCENT - mnC3;
                     }
                 break;
diff --git a/oox/source/drawingml/customshapeproperties.cxx b/oox/source/drawingml/customshapeproperties.cxx
index 53cbf95844ea..7e87ea8d5248 100644
--- a/oox/source/drawingml/customshapeproperties.cxx
+++ b/oox/source/drawingml/customshapeproperties.cxx
@@ -189,25 +189,24 @@ void CustomShapeProperties::pushToPropSet(
                         if ( aGeoPropSeq[ i ].Value >>= aAdjustmentSeq )
                         {
                             int nIndex=0;
-                            for (std::vector< CustomShapeGuide >::const_iterator aIter( maAdjustmentGuideList.begin() ), aEnd(maAdjustmentGuideList.end());
-                             aIter != aEnd; ++aIter)
+                            for (auto const& adjustmentGuide : maAdjustmentGuideList)
                             {
-                                if ( (*aIter).maName.getLength() > 3 )
+                                if ( adjustmentGuide.maName.getLength() > 3 )
                                 {
-                                    sal_Int32 nAdjustmentIndex = (*aIter).maName.copy( 3 ).toInt32() - 1;
+                                    sal_Int32 nAdjustmentIndex = adjustmentGuide.maName.copy( 3 ).toInt32() - 1;
                                     if ( ( nAdjustmentIndex >= 0 ) && ( nAdjustmentIndex < aAdjustmentSeq.getLength() ) )
                                     {
                                         EnhancedCustomShapeAdjustmentValue aAdjustmentVal;
-                                        aAdjustmentVal.Value <<= (*aIter).maFormula.toInt32();
+                                        aAdjustmentVal.Value <<= adjustmentGuide.maFormula.toInt32();
                                         aAdjustmentVal.State = PropertyState_DIRECT_VALUE;
-                                        aAdjustmentVal.Name = (*aIter).maName;
+                                        aAdjustmentVal.Name = adjustmentGuide.maName;
                                         aAdjustmentSeq[ nAdjustmentIndex ] = aAdjustmentVal;
                                     }
                                 } else if ( aAdjustmentSeq.getLength() > 0 ) {
                                     EnhancedCustomShapeAdjustmentValue aAdjustmentVal;
-                                    aAdjustmentVal.Value <<= (*aIter).maFormula.toInt32();
+                                    aAdjustmentVal.Value <<= adjustmentGuide.maFormula.toInt32();
                                     aAdjustmentVal.State = PropertyState_DIRECT_VALUE;
-                                    aAdjustmentVal.Name = (*aIter).maName;
+                                    aAdjustmentVal.Name = adjustmentGuide.maName;
                                     aAdjustmentSeq[ nIndex++ ] = aAdjustmentVal;
                                 }
                             }
diff --git a/oox/source/drawingml/diagram/diagram.cxx b/oox/source/drawingml/diagram/diagram.cxx
index e7696898c81c..e7a2df78bd5e 100644
--- a/oox/source/drawingml/diagram/diagram.cxx
+++ b/oox/source/drawingml/diagram/diagram.cxx
@@ -115,20 +115,17 @@ static sal_Int32 calcDepth( const OUString& rNodeName,
                             const dgm::Connections& rCnx )
 {
     // find length of longest path in 'isChild' graph, ending with rNodeName
-    dgm::Connections::const_iterator aCurrCxn( rCnx.begin() );
-    const dgm::Connections::const_iterator aEndCxn( rCnx.end() );
-    while( aCurrCxn != aEndCxn )
+    for (auto const& elem : rCnx)
     {
-        if( !aCurrCxn->msParTransId.isEmpty() &&
-            !aCurrCxn->msSibTransId.isEmpty() &&
-            !aCurrCxn->msSourceId.isEmpty() &&
-            !aCurrCxn->msDestId.isEmpty() &&
-            aCurrCxn->mnType == XML_parOf &&
-            rNodeName == aCurrCxn->msDestId )
+        if( !elem.msParTransId.isEmpty() &&
+            !elem.msSibTransId.isEmpty() &&
+            !elem.msSourceId.isEmpty() &&
+            !elem.msDestId.isEmpty() &&
+            elem.mnType == XML_parOf &&
+            rNodeName == elem.msDestId )
         {
-            return calcDepth(aCurrCxn->msSourceId, rCnx) + 1;
+            return calcDepth(elem.msSourceId, rCnx) + 1;
         }
-        ++aCurrCxn;
     }
 
     return 0;
@@ -143,27 +140,25 @@ void Diagram::build(  )
     output << "digraph datatree {" << std::endl;
 #endif
     dgm::Points& rPoints = getData()->getPoints();
-    dgm::Points::iterator aCurrPoint(rPoints.begin());
-    dgm::Points::iterator aEndPoint(rPoints.end());
-    while( aCurrPoint != aEndPoint )
+    for (auto & point : rPoints)
     {
 #ifdef DEBUG_OOX_DIAGRAM
         output << "\t"
-               << normalizeDotName(aCurrPoint->msModelId).getStr()
+               << normalizeDotName(point.msModelId).getStr()
                << "[";
 
-        if( !aCurrPoint->msPresentationLayoutName.isEmpty() )
+        if( !point.msPresentationLayoutName.isEmpty() )
             output << "label=\""
                    << OUStringToOString(
-                       aCurrPoint->msPresentationLayoutName,
+                       point.msPresentationLayoutName,
                        RTL_TEXTENCODING_UTF8).getStr() << "\", ";
         else
             output << "label=\""
                    << OUStringToOString(
-                       aCurrPoint->msModelId,
+                       point.msModelId,
                        RTL_TEXTENCODING_UTF8).getStr() << "\", ";
 
-        switch( aCurrPoint->mnType )
+        switch( point.mnType )
         {
             case XML_doc: output << "style=filled, color=red"; break;
             case XML_asst: output << "style=filled, color=green"; break;
@@ -178,10 +173,10 @@ void Diagram::build(  )
 #endif
 
         // does currpoint have any text set?
-        if( aCurrPoint->mpShape &&
-            aCurrPoint->mpShape->getTextBody() &&
-            !aCurrPoint->mpShape->getTextBody()->getParagraphs().empty() &&
-            !aCurrPoint->mpShape->getTextBody()->getParagraphs().front()->getRuns().empty() )
+        if( point.mpShape &&
+            point.mpShape->getTextBody() &&
+            !point.mpShape->getTextBody()->getParagraphs().empty() &&
+            !point.mpShape->getTextBody()->getParagraphs().front()->getRuns().empty() )
         {
 #ifdef DEBUG_OOX_DIAGRAM
             static sal_Int32 nCount=0;
@@ -190,11 +185,11 @@ void Diagram::build(  )
                    << " ["
                    << "label=\""
                    << OUStringToOString(
-                       aCurrPoint->mpShape->getTextBody()->getParagraphs().front()->getRuns().front()->getText(),
+                       point.mpShape->getTextBody()->getParagraphs().front()->getRuns().front()->getText(),
                        RTL_TEXTENCODING_UTF8).getStr()
                    << "\"" << "];" << std::endl;
             output << "\t"
-                   << normalizeDotName(aCurrPoint->msModelId).getStr()
+                   << normalizeDotName(point.msModelId).getStr()
                    << " -> "
                    << "textNode" << nCount++
                    << ";" << std::endl;
@@ -202,107 +197,95 @@ void Diagram::build(  )
         }
 
         const bool bInserted1=getData()->getPointNameMap().insert(
-            std::make_pair(aCurrPoint->msModelId,&(*aCurrPoint))).second;
+            std::make_pair(point.msModelId,&point)).second;
 
         SAL_WARN_IF(!bInserted1, "oox.drawingml", "Diagram::build(): non-unique point model id");
 
-        if( !aCurrPoint->msPresentationLayoutName.isEmpty() )
+        if( !point.msPresentationLayoutName.isEmpty() )
         {
             DiagramData::PointsNameMap::value_type::second_type& rVec=
-                getData()->getPointsPresNameMap()[aCurrPoint->msPresentationLayoutName];
-            rVec.push_back(&(*aCurrPoint));
+                getData()->getPointsPresNameMap()[point.msPresentationLayoutName];
+            rVec.push_back(&point);
         }
-        ++aCurrPoint;
     }
 
     const dgm::Connections& rConnections = getData()->getConnections();
-    dgm::Connections::const_iterator aCurrCxn(rConnections.begin());
-    const dgm::Connections::const_iterator aEndCxn(rConnections.end());
-    while( aCurrCxn != aEndCxn )
+    for (auto const& connection : rConnections)
     {
 #ifdef DEBUG_OOX_DIAGRAM
-        if( !aCurrCxn->msParTransId.isEmpty() ||
-            !aCurrCxn->msSibTransId.isEmpty() )
+        if( !connection.msParTransId.isEmpty() ||
+            !connection.msSibTransId.isEmpty() )
         {
-            if( !aCurrCxn->msSourceId.isEmpty() ||
-                !aCurrCxn->msDestId.isEmpty() )
+            if( !connection.msSourceId.isEmpty() ||
+                !connection.msDestId.isEmpty() )
             {
                 output << "\t"
-                       << normalizeDotName(aCurrCxn->msSourceId).getStr()
+                       << normalizeDotName(connection.msSourceId).getStr()
                        << " -> "
-                       << normalizeDotName(aCurrCxn->msParTransId).getStr()
+                       << normalizeDotName(connection.msParTransId).getStr()
                        << " -> "
-                       << normalizeDotName(aCurrCxn->msSibTransId).getStr()
+                       << normalizeDotName(connection.msSibTransId).getStr()
                        << " -> "
-                       << normalizeDotName(aCurrCxn->msDestId).getStr()
+                       << normalizeDotName(connection.msDestId).getStr()
                        << " [style=dotted,"
-                       << ((aCurrCxn->mnType == XML_presOf) ? " color=red, " : ((aCurrCxn->mnType == XML_presParOf) ? " color=green, " : " "))
+                       << ((connection.mnType == XML_presOf) ? " color=red, " : ((connection.mnType == XML_presParOf) ? " color=green, " : " "))
                        << "label=\""
-                       << OUStringToOString(aCurrCxn->msModelId,
+                       << OUStringToOString(connection.msModelId,
                                                  RTL_TEXTENCODING_UTF8 ).getStr()
                        << "\"];" << std::endl;
             }
             else
             {
                 output << "\t"
-                       << normalizeDotName(aCurrCxn->msParTransId).getStr()
+                       << normalizeDotName(connection.msParTransId).getStr()
                        << " -> "
-                       << normalizeDotName(aCurrCxn->msSibTransId).getStr()
+                       << normalizeDotName(connection.msSibTransId).getStr()
                        << " ["
-                       << ((aCurrCxn->mnType == XML_presOf) ? " color=red, " : ((aCurrCxn->mnType == XML_presParOf) ? " color=green, " : " "))
+                       << ((connection.mnType == XML_presOf) ? " color=red, " : ((connection.mnType == XML_presParOf) ? " color=green, " : " "))
                        << "label=\""
-                       << OUStringToOString(aCurrCxn->msModelId,
+                       << OUStringToOString(connection.msModelId,
                                                  RTL_TEXTENCODING_UTF8 ).getStr()
                        << "\"];" << std::endl;
             }
         }
-        else if( !aCurrCxn->msSourceId.isEmpty() ||
-                 !aCurrCxn->msDestId.isEmpty() )
+        else if( !connection.msSourceId.isEmpty() ||
+                 !connection.msDestId.isEmpty() )
             output << "\t"
-                   << normalizeDotName(aCurrCxn->msSourceId).getStr()
+                   << normalizeDotName(connection.msSourceId).getStr()
                    << " -> "
-                   << normalizeDotName(aCurrCxn->msDestId).getStr()
+                   << normalizeDotName(connection.msDestId).getStr()
                    << " [label=\""
-                   << OUStringToOString(aCurrCxn->msModelId,
+                   << OUStringToOString(connection.msModelId,
                                              RTL_TEXTENCODING_UTF8 ).getStr()
-                   << ((aCurrCxn->mnType == XML_presOf) ? "\", color=red]" : ((aCurrCxn->mnType == XML_presParOf) ? "\", color=green]" : "\"]"))
+                   << ((connection.mnType == XML_presOf) ? "\", color=red]" : ((connection.mnType == XML_presParOf) ? "\", color=green]" : "\"]"))
                    << ";" << std::endl;
 #endif
 
         const bool bInserted1=getData()->getConnectionNameMap().insert(
-            std::make_pair(aCurrCxn->msModelId,&(*aCurrCxn))).second;
+            std::make_pair(connection.msModelId,&connection)).second;
 
         SAL_WARN_IF(!bInserted1, "oox.drawingml", "Diagram::build(): non-unique connection model id");
 
-        if( aCurrCxn->mnType == XML_presOf )
+        if( connection.mnType == XML_presOf )
         {
-            DiagramData::StringMap::value_type::second_type& rVec=getData()->getPresOfNameMap()[aCurrCxn->msDestId];
+            DiagramData::StringMap::value_type::second_type& rVec=getData()->getPresOfNameMap()[connection.msDestId];
             rVec.emplace_back(
-                    aCurrCxn->msSourceId,sal_Int32(0));
+                    connection.msSourceId,sal_Int32(0));
         }
-
-        ++aCurrCxn;
     }
 
     // assign outline levels
     DiagramData::StringMap& rStringMap = getData()->getPresOfNameMap();
-    DiagramData::StringMap::iterator aPresOfIter=rStringMap.begin();
-    const DiagramData::StringMap::iterator aPresOfEnd=rStringMap.end();
-    while( aPresOfIter != aPresOfEnd )
+    for (auto & elemPresOf : rStringMap)
     {
-        DiagramData::StringMap::value_type::second_type::iterator aPresOfNodeIterCalcLevel=aPresOfIter->second.begin();
-        const DiagramData::StringMap::value_type::second_type::iterator aPresOfNodeEnd=aPresOfIter->second.end();
-        while(aPresOfNodeIterCalcLevel != aPresOfNodeEnd)
+        for (auto & elem : elemPresOf.second)
         {
-            const sal_Int32 nDepth=calcDepth(aPresOfNodeIterCalcLevel->first,
+            const sal_Int32 nDepth=calcDepth(elem.first,
                                              getData()->getConnections());
-            aPresOfNodeIterCalcLevel->second = nDepth != 0 ? nDepth : -1;
+            elem.second = nDepth != 0 ? nDepth : -1;
             if (nDepth > getData()->getMaxDepth())
                 getData()->setMaxDepth(nDepth);
-            ++aPresOfNodeIterCalcLevel;
         }
-
-        ++aPresOfIter;
     }
 #ifdef DEBUG_OOX_DIAGRAM
     output << "}" << std::endl;
@@ -343,12 +326,10 @@ uno::Sequence<beans::PropertyValue> Diagram::getDomsAsPropertyValues() const
 
     uno::Sequence<beans::PropertyValue> aValue(length);
     beans::PropertyValue* pValue = aValue.getArray();
-    for (DiagramDomMap::const_iterator i = maMainDomMap.begin();
-         i != maMainDomMap.end();
-         ++i)
+    for (auto const& mainDom : maMainDomMap)
     {
-        pValue->Name = i->first;
-        pValue->Value <<= i->second;
+        pValue->Name = mainDom.first;
+        pValue->Value <<= mainDom.second;
         ++pValue;
     }
 
@@ -425,9 +406,8 @@ void loadDiagram( ShapePtr const & pShape,
                 xRefDataModel->getFragmentPath(), "image" );
 
         // Pass the info to pShape
-        for( ::std::vector<OUString>::const_iterator aIt = pData->getExtDrawings().begin(), aEnd = pData->getExtDrawings().end();
-                aIt != aEnd; ++aIt )
-                pShape->addExtDrawingRelId( *aIt );
+        for (auto const& extDrawing : pData->getExtDrawings())
+                pShape->addExtDrawingRelId(extDrawing);
     }
 
     // extLst is present, lets bet on that and ignore the rest of the data from here
diff --git a/oox/source/drawingml/effectproperties.cxx b/oox/source/drawingml/effectproperties.cxx
index 602f6911a5e4..49f1d915a11c 100644
--- a/oox/source/drawingml/effectproperties.cxx
+++ b/oox/source/drawingml/effectproperties.cxx
@@ -80,10 +80,10 @@ css::beans::PropertyValue Effect::getEffect()
 
     css::uno::Sequence< css::beans::PropertyValue > aSeq( maAttribs.size() );
     sal_uInt32 i = 0;
-    for( std::map< OUString, css::uno::Any >::iterator it = maAttribs.begin(); it != maAttribs.end(); ++it )
+    for (auto const& attrib : maAttribs)
     {
-        aSeq[i].Name = it->first;
-        aSeq[i].Value = it->second;
+        aSeq[i].Name = attrib.first;
+        aSeq[i].Value = attrib.second;
         i++;
     }
 
diff --git a/oox/source/drawingml/fillproperties.cxx b/oox/source/drawingml/fillproperties.cxx
index 9351b473068f..944a4dbee05d 100644
--- a/oox/source/drawingml/fillproperties.cxx
+++ b/oox/source/drawingml/fillproperties.cxx
@@ -429,13 +429,12 @@ void FillProperties::pushToPropMap( ShapePropertyMap& rPropMap,
 
                         SAL_INFO("oox.drawingml.gradient", "symmetric: " << (bSymmetric ? "YES" : "NO") <<
                                  ", number of stops: " << aGradientStops.size());
-                        for (GradientFillProperties::GradientStopMap::iterator p(aGradientStops.begin());
-                             p != aGradientStops.end();
-                             ++p)
-                            SAL_INFO("oox.drawingml.gradient", "  " << std::distance(aGradientStops.begin(), p) << ": " <<
-                                     p->first << ": " <<
-                                     std::hex << sal_Int32(p->second.getColor( rGraphicHelper, nPhClr )) << std::dec <<
-                                     "@" << (100-p->second.getTransparency()) << "%");
+                        size_t nIndex = 0;
+                        for (auto const& gradientStop : aGradientStops)
+                            SAL_INFO("oox.drawingml.gradient", "  " << nIndex++ << ": " <<
+                                     gradientStop.first << ": " <<
+                                     std::hex << sal_Int32(gradientStop.second.getColor( rGraphicHelper, nPhClr )) << std::dec <<
+                                     "@" << (100 - gradientStop.second.getTransparency()) << "%");
 
                         // Now estimate the simple LO style gradient (only two stops, at n% and 100%, where n ==
                         // the "border") that best emulates the gradient between begin() and prior(end()).
diff --git a/oox/source/drawingml/shape.cxx b/oox/source/drawingml/shape.cxx
index b3c48b0b67c7..5593e9656e9b 100644
--- a/oox/source/drawingml/shape.cxx
+++ b/oox/source/drawingml/shape.cxx
@@ -429,16 +429,14 @@ Reference< XShape > const & Shape::createAndInsert(
     if ( mpTablePropertiesPtr.get() && rServiceName == "com.sun.star.drawing.TableShape" )
     {
         maSize.Width = 0;
-        for( std::vector< sal_Int32 >::const_iterator aTableColIter(mpTablePropertiesPtr->getTableGrid().begin());
-             aTableColIter != mpTablePropertiesPtr->getTableGrid().end(); ++aTableColIter )
+        for (auto const& elem : mpTablePropertiesPtr->getTableGrid())
         {
-            maSize.Width += *aTableColIter;
+            maSize.Width += elem;
         }
         maSize.Height = 0;
-        for( std::vector< ::oox::drawingml::table::TableRow >::const_iterator aTableRowIter(mpTablePropertiesPtr->getTableRows().begin());
-             aTableRowIter != mpTablePropertiesPtr->getTableRows().end(); ++aTableRowIter )
+        for (auto const& elem : mpTablePropertiesPtr->getTableRows())
         {
-            maSize.Height += (*aTableRowIter).getHeight();
+            maSize.Height += elem.getHeight();
         }
     }
 
@@ -1568,11 +1566,11 @@ uno::Sequence< uno::Sequence< uno::Any > >  Shape::resolveRelationshipsOfTypeFro
         if ( xImageRels )
         {
             xRelListTemp.realloc( xImageRels->size() );
-            for( ::std::map< OUString, core::Relation >::const_iterator aIt = xImageRels->begin(), aEnd = xImageRels->end(); aIt != aEnd; ++aIt )
+            for (auto const& imageRel : *xImageRels)
             {
                 uno::Sequence< uno::Any > diagramRelTuple (3);
                 // [0] => RID, [1] => InputStream [2] => extension
-                OUString sRelId = aIt->second.maId;
+                OUString sRelId = imageRel.second.maId;
 
                 diagramRelTuple[0] <<= sRelId;
                 OUString sTarget = xImageRels->getFragmentPathFromRelId( sRelId );
diff --git a/oox/source/drawingml/table/tableproperties.cxx b/oox/source/drawingml/table/tableproperties.cxx
index 527b65829d17..2c92390772e8 100644
--- a/oox/source/drawingml/table/tableproperties.cxx
+++ b/oox/source/drawingml/table/tableproperties.cxx
@@ -249,15 +249,13 @@ const TableStyle& TableProperties::getUsedTableStyle( const ::oox::core::XmlFilt
         const std::vector< TableStyle >& rTableStyles( rBase.getTableStyles()->getTableStyles() );
         const OUString aStyleId( getStyleId() );
 
-        std::vector< TableStyle >::const_iterator aIter( rTableStyles.begin() );
-        while( aIter != rTableStyles.end() )
+        for (auto const& tableStyle : rTableStyles)
         {
-            if ( const_cast< TableStyle& >( *aIter ).getStyleId() == aStyleId )
+            if ( const_cast< TableStyle& >(tableStyle).getStyleId() == aStyleId )
             {
-                pTableStyle = &const_cast< TableStyle& >( *aIter );
+                pTableStyle = &const_cast< TableStyle& >(tableStyle);
                 break;  // we get the correct style
             }
-            ++aIter;
         }
         //if the pptx just has table style id, but no table style content, we will create the table style ourselves
         if (!pTableStyle)
@@ -285,16 +283,12 @@ void TableProperties::pushToPropSet( const ::oox::core::XmlFilterBase& rFilterBa
     TableStyle* pTableStyleToDelete = nullptr;
     const TableStyle& rTableStyle( getUsedTableStyle( rFilterBase, pTableStyleToDelete ) );
     sal_Int32 nRow = 0;
-    const std::vector< TableRow >::const_iterator aTableRowEnd( mvTableRows.end() );
-    for (std::vector< TableRow >::iterator aTableRowIter( mvTableRows.begin() );
-         aTableRowIter != aTableRowEnd ; ++aTableRowIter, ++nRow)
+    for (auto & tableRow : mvTableRows)
     {
         sal_Int32 nColumn = 0;
-        const std::vector< TableCell >::const_iterator aTableCellEnd( aTableRowIter->getTableCells().end() );
-        for (std::vector< TableCell >::iterator aTableCellIter( aTableRowIter->getTableCells().begin() );
-            aTableCellIter != aTableCellEnd ; ++aTableCellIter, ++nColumn)
+        for (auto & tableCell : tableRow.getTableCells())
         {
-            TableCell& rTableCell( *aTableCellIter );
+            TableCell& rTableCell(tableCell);
             if ( !rTableCell.getvMerge() && !rTableCell.gethMerge() )
             {
                 uno::Reference< XTable > xTable( xColumnRowRange, uno::UNO_QUERY_THROW );
@@ -303,9 +297,11 @@ void TableProperties::pushToPropSet( const ::oox::core::XmlFilterBase& rFilterBa
 
                 Reference< XCellRange > xCellRange( xTable, UNO_QUERY_THROW );
                 rTableCell.pushToXCell( rFilterBase, pMasterTextListStyle, xCellRange->getCellByPosition( nColumn, nRow ), *this, rTableStyle,
-                    nColumn, aTableRowIter->getTableCells().size()-1, nRow, mvTableRows.size()-1 );
+                    nColumn, tableRow.getTableCells().size()-1, nRow, mvTableRows.size()-1 );
             }
+            ++nColumn;
         }
+        ++nRow;
     }
 
     delete pTableStyleToDelete;


More information about the Libreoffice-commits mailing list