Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix clazy unnecessary unnecessary container allocation warnings
  • Loading branch information
nyalldawson committed Feb 2, 2018
1 parent ef7e7c5 commit 252dbdc
Show file tree
Hide file tree
Showing 8 changed files with 29 additions and 27 deletions.
Expand Up @@ -176,10 +176,10 @@ QVariantMap QgsRasterLayerUniqueValuesReportAlgorithm::processAlgorithm( const Q
out << QObject::tr( "<p>%1: %2</p>\n" ).arg( QObject::tr( "NODATA pixel count" ) ).arg( noDataCount );
out << QString( "<table><tr><td>%1</td><td>%2</td><td>%3 (%4)</td></tr>\n" ).arg( QObject::tr( "Value" ), QObject::tr( "Pixel count" ), QObject::tr( "Area" ), areaUnit );

for ( double key : sortedUniqueValues.keys() )
for ( auto it = sortedUniqueValues.constBegin(); it != sortedUniqueValues.constEnd(); ++it )
{
double area = sortedUniqueValues[key] * pixelArea;
out << QString( "<tr><td>%1</td><td>%2</td><td>%3</td></tr>\n" ).arg( key ).arg( sortedUniqueValues[key] ).arg( QString::number( area, 'g', 16 ) );
double area = it.value() * pixelArea;
out << QString( "<tr><td>%1</td><td>%2</td><td>%3</td></tr>\n" ).arg( it.key() ).arg( it.value() ).arg( QString::number( area, 'g', 16 ) );
}
out << QString( "</table>\n</body></html>" );
outputs.insert( QStringLiteral( "OUTPUT_HTML_FILE" ), outputFile );
Expand Down
10 changes: 5 additions & 5 deletions src/app/layout/qgslayoutlegendwidget.cpp
Expand Up @@ -739,20 +739,20 @@ void QgsLayoutLegendWidget::mRemoveToolButton_clicked()
nodesWithRemoval[nodeLayer].append( _unfilteredLegendNodeIndex( legendNode ) );
}
}
Q_FOREACH ( QgsLayerTreeLayer *nodeLayer, nodesWithRemoval.keys() )
for ( auto it = nodesWithRemoval.constBegin(); it != nodesWithRemoval.constEnd(); ++it )
{
QList<int> toDelete = nodesWithRemoval[nodeLayer];
QList<int> toDelete = it.value();
std::sort( toDelete.begin(), toDelete.end(), std::greater<int>() );
QList<int> order = QgsMapLayerLegendUtils::legendNodeOrder( nodeLayer );
QList<int> order = QgsMapLayerLegendUtils::legendNodeOrder( it.key() );

Q_FOREACH ( int i, toDelete )
{
if ( i >= 0 && i < order.count() )
order.removeAt( i );
}

QgsMapLayerLegendUtils::setLegendNodeOrder( nodeLayer, order );
mItemTreeView->layerTreeModel()->refreshLayerLegend( nodeLayer );
QgsMapLayerLegendUtils::setLegendNodeOrder( it.key(), order );
mItemTreeView->layerTreeModel()->refreshLayerLegend( it.key() );
}

// then remove layer tree nodes
Expand Down
5 changes: 3 additions & 2 deletions src/core/layout/qgscompositionconverter.cpp
Expand Up @@ -1655,8 +1655,9 @@ bool QgsCompositionConverter::readOldComposerObjectXml( QgsLayoutObject *layoutI

void QgsCompositionConverter::readOldDataDefinedPropertyMap( const QDomElement &itemElem, QgsPropertyCollection &dataDefinedProperties )
{
QgsPropertiesDefinition::const_iterator i = QgsCompositionConverter::propertyDefinitions().constBegin();
for ( ; i != QgsCompositionConverter::propertyDefinitions().constEnd(); ++i )
const QgsPropertiesDefinition defs = QgsCompositionConverter::propertyDefinitions();
QgsPropertiesDefinition::const_iterator i = defs.constBegin();
for ( ; i != defs.constEnd(); ++i )
{
QString elemName = i.value().name();
QDomNodeList ddNodeList = itemElem.elementsByTagName( elemName );
Expand Down
18 changes: 9 additions & 9 deletions src/core/qgsauxiliarystorage.cpp
Expand Up @@ -428,11 +428,11 @@ QgsPropertyDefinition QgsAuxiliaryLayer::propertyDefinitionFromField( const QgsF
if ( origin.compare( "labeling", Qt::CaseInsensitive ) == 0 )
{
const QgsPropertiesDefinition props = QgsPalLayerSettings::propertyDefinitions();
for ( const QgsPropertyDefinition &p : props.values() )
for ( auto it = props.constBegin(); it != props.constEnd(); ++it )
{
if ( p.name().compare( propertyName, Qt::CaseInsensitive ) == 0 )
if ( it.value().name().compare( propertyName, Qt::CaseInsensitive ) == 0 )
{
def = p;
def = it.value();
if ( parts.size() == 3 )
def.setComment( parts[2] );
break;
Expand All @@ -442,11 +442,11 @@ QgsPropertyDefinition QgsAuxiliaryLayer::propertyDefinitionFromField( const QgsF
else if ( origin.compare( "symbol", Qt::CaseInsensitive ) == 0 )
{
const QgsPropertiesDefinition props = QgsSymbolLayer::propertyDefinitions();
for ( const QgsPropertyDefinition &p : props.values() )
for ( auto it = props.constBegin(); it != props.constEnd(); ++it )
{
if ( p.name().compare( propertyName, Qt::CaseInsensitive ) == 0 )
if ( it.value().name().compare( propertyName, Qt::CaseInsensitive ) == 0 )
{
def = p;
def = it.value();
if ( parts.size() == 3 )
def.setComment( parts[2] );
break;
Expand All @@ -456,11 +456,11 @@ QgsPropertyDefinition QgsAuxiliaryLayer::propertyDefinitionFromField( const QgsF
else if ( origin.compare( "diagram", Qt::CaseInsensitive ) == 0 )
{
const QgsPropertiesDefinition props = QgsDiagramLayerSettings::propertyDefinitions();
for ( const QgsPropertyDefinition &p : props.values() )
for ( auto it = props.constBegin(); it != props.constEnd(); ++it )
{
if ( p.name().compare( propertyName, Qt::CaseInsensitive ) == 0 )
if ( it.value().name().compare( propertyName, Qt::CaseInsensitive ) == 0 )
{
def = p;
def = it.value();
if ( parts.size() == 3 )
def.setComment( parts[2] );
break;
Expand Down
7 changes: 4 additions & 3 deletions src/core/qgsproject.cpp
Expand Up @@ -2414,12 +2414,13 @@ void QgsProject::setTrustLayerMetadata( bool trust )

bool QgsProject::saveAuxiliaryStorage( const QString &filename )
{
for ( QgsMapLayer *l : mapLayers().values() )
const QMap<QString, QgsMapLayer *> layers = mapLayers();
for ( auto it = layers.constBegin(); it != layers.constEnd(); ++it )
{
if ( l->type() != QgsMapLayer::VectorLayer )
if ( it.value()->type() != QgsMapLayer::VectorLayer )
continue;

QgsVectorLayer *vl = qobject_cast<QgsVectorLayer *>( l );
QgsVectorLayer *vl = qobject_cast<QgsVectorLayer *>( it.value() );
if ( vl && vl->auxiliaryLayer() )
{
vl->auxiliaryLayer()->save();
Expand Down
2 changes: 1 addition & 1 deletion src/gui/layout/qgslayoutmousehandles.cpp
Expand Up @@ -296,7 +296,7 @@ bool QgsLayoutMouseHandles::selectionRotation( double &rotation ) const
double firstItemRotation = ( *itemIter )->rotation();

//iterate through remaining items, checking if they have same rotation
for ( ++itemIter; itemIter != selectedItems.end(); ++itemIter )
for ( ++itemIter; itemIter != selectedItems.constEnd(); ++itemIter )
{
if ( !qgsDoubleNear( ( *itemIter )->rotation(), firstItemRotation ) )
{
Expand Down
2 changes: 1 addition & 1 deletion src/providers/wcs/qgswcssourceselect.cpp
Expand Up @@ -87,7 +87,7 @@ void QgsWCSSourceSelect::populateLayerList()
lItem->setData( 0, Qt::UserRole + 1, "" );

// Make only leaves selectable
if ( !coverageParents.keys( coverage->orderId ).isEmpty() )
if ( coverageParents.contains( coverage->orderId ) )
{
lItem->setFlags( Qt::ItemIsEnabled );
}
Expand Down
6 changes: 3 additions & 3 deletions tests/src/core/testqgsrasterlayer.cpp
Expand Up @@ -279,9 +279,9 @@ void TestQgsRasterLayer::populateColorRampShader( QgsColorRampShader *colorRampS

//items to imitate old pseudo color renderer
QList<QgsColorRampShader::ColorRampItem> colorRampItems;
QList<double>::const_iterator value_it = entryValues.begin();
QVector<QColor>::const_iterator color_it = entryColors.begin();
for ( ; value_it != entryValues.end(); ++value_it, ++color_it )
QList<double>::const_iterator value_it = entryValues.constBegin();
QVector<QColor>::const_iterator color_it = entryColors.constBegin();
for ( ; value_it != entryValues.constEnd(); ++value_it, ++color_it )
{
colorRampItems.append( QgsColorRampShader::ColorRampItem( *value_it, *color_it ) );
}
Expand Down

0 comments on commit 252dbdc

Please sign in to comment.