Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
avoid extra const copy
  • Loading branch information
alexbruy committed Mar 15, 2020
1 parent 8842ab3 commit 0e652a1
Showing 1 changed file with 11 additions and 19 deletions.
30 changes: 11 additions & 19 deletions src/core/processing/qgsprocessingparameters.cpp
Expand Up @@ -2277,8 +2277,7 @@ QString QgsProcessingParameterMapLayer::asScriptCode() const
code += QStringLiteral( "optional " );
code += QStringLiteral( "layer " );

const auto constMDataTypes = mDataTypes;
for ( int type : constMDataTypes )
for ( int type : mDataTypes )
{
switch ( type )
{
Expand Down Expand Up @@ -2396,8 +2395,7 @@ QVariantMap QgsProcessingParameterMapLayer::toVariantMap() const
{
QVariantMap map = QgsProcessingParameterDefinition::toVariantMap();
QVariantList types;
const auto constMDataTypes = mDataTypes;
for ( int type : constMDataTypes )
for ( int type : mDataTypes )
{
types << type;
}
Expand All @@ -2409,9 +2407,8 @@ bool QgsProcessingParameterMapLayer::fromVariantMap( const QVariantMap &map )
{
QgsProcessingParameterDefinition::fromVariantMap( map );
mDataTypes.clear();
QVariantList values = map.value( QStringLiteral( "data_types" ) ).toList();
const auto constValues = values;
for ( const QVariant &val : constValues )
const QVariantList values = map.value( QStringLiteral( "data_types" ) ).toList();
for ( const QVariant &val : values )
{
mDataTypes << val.toInt();
}
Expand Down Expand Up @@ -4060,8 +4057,7 @@ QVariantMap QgsProcessingParameterVectorLayer::toVariantMap() const
{
QVariantMap map = QgsProcessingParameterDefinition::toVariantMap();
QVariantList types;
const auto constMDataTypes = mDataTypes;
for ( int type : constMDataTypes )
for ( int type : mDataTypes )
{
types << type;
}
Expand All @@ -4073,9 +4069,8 @@ bool QgsProcessingParameterVectorLayer::fromVariantMap( const QVariantMap &map )
{
QgsProcessingParameterDefinition::fromVariantMap( map );
mDataTypes.clear();
QVariantList values = map.value( QStringLiteral( "data_types" ) ).toList();
const auto constValues = values;
for ( const QVariant &val : constValues )
const QVariantList values = map.value( QStringLiteral( "data_types" ) ).toList();
for ( const QVariant &val : values )
{
mDataTypes << val.toInt();
}
Expand Down Expand Up @@ -4569,8 +4564,7 @@ QString QgsProcessingParameterFeatureSource::asScriptCode() const
code += QStringLiteral( "optional " );
code += QStringLiteral( "source " );

const auto constMDataTypes = mDataTypes;
for ( int type : constMDataTypes )
for ( int type : mDataTypes )
{
switch ( type )
{
Expand Down Expand Up @@ -4630,8 +4624,7 @@ QVariantMap QgsProcessingParameterFeatureSource::toVariantMap() const
{
QVariantMap map = QgsProcessingParameterDefinition::toVariantMap();
QVariantList types;
const auto constMDataTypes = mDataTypes;
for ( int type : constMDataTypes )
for ( int type : mDataTypes )
{
types << type;
}
Expand All @@ -4643,9 +4636,8 @@ bool QgsProcessingParameterFeatureSource::fromVariantMap( const QVariantMap &map
{
QgsProcessingParameterDefinition::fromVariantMap( map );
mDataTypes.clear();
QVariantList values = map.value( QStringLiteral( "data_types" ) ).toList();
const auto constValues = values;
for ( const QVariant &val : constValues )
const QVariantList values = map.value( QStringLiteral( "data_types" ) ).toList();
for ( const QVariant &val : values )
{
mDataTypes << val.toInt();
}
Expand Down

0 comments on commit 0e652a1

Please sign in to comment.