Merge pull request #6862 from m-kuhn/core-headers-no-qforeach

m-kuhn · web-flow · commit f8165888e45c · 2018-04-26T11:42:52.000+02:00
Rewrite all core header usages of Q_FOREACH
diff --git a/python/core/qgsrelation.sip.in b/python/core/qgsrelation.sip.in
@@ -238,7 +238,7 @@ The second element of each pair are the field names of the matching primary key.
 %MethodCode
     const QList< QgsRelation::FieldPair > &pairs = sipCpp->fieldPairs();
     sipRes = new QMap< QString, QString >();
-    Q_FOREACH ( const QgsRelation::FieldPair &pair, pairs )
+    for ( const QgsRelation::FieldPair &pair : pairs )
     {
       sipRes->insert( pair.first, pair.second );
     }
diff --git a/src/core/qgsexpressionsorter.h b/src/core/qgsexpressionsorter.h
@@ -36,7 +36,7 @@ class QgsExpressionSorter
     bool operator()( const QgsIndexedFeature &f1, const QgsIndexedFeature &f2 ) const
     {
       int i = 0;
-      Q_FOREACH ( const QgsFeatureRequest::OrderByClause &orderBy, mPreparedOrderBys )
+      for ( const QgsFeatureRequest::OrderByClause &orderBy : qgis::as_const( mPreparedOrderBys ) )
       {
         const QVariant &v1 = f1.mIndexes.at( i );
         const QVariant &v2 = f2.mIndexes.at( i );
@@ -143,15 +143,15 @@ class QgsExpressionSorter
 
       QgsIndexedFeature indexedFeature;
 
-      Q_FOREACH ( const QgsFeature &f, features )
+      for ( const QgsFeature &f : qgis::as_const( features ) )
       {
         indexedFeature.mIndexes.resize( mPreparedOrderBys.size() );
         indexedFeature.mFeature = f;
 
         expressionContext->setFeature( indexedFeature.mFeature );
 
         int i = 0;
-        Q_FOREACH ( const QgsFeatureRequest::OrderByClause &orderBy, mPreparedOrderBys )
+        for ( const QgsFeatureRequest::OrderByClause &orderBy : qgis::as_const( mPreparedOrderBys ) )
         {
           indexedFeature.mIndexes.replace( i++, orderBy.expression().evaluate( expressionContext ) );
         }
@@ -164,7 +164,7 @@ class QgsExpressionSorter
 
       features.clear();
 
-      Q_FOREACH ( const QgsIndexedFeature &indexedFeature, indexedFeatures )
+      for ( const QgsIndexedFeature &indexedFeature : qgis::as_const( indexedFeatures ) )
         features.append( indexedFeature.mFeature );
     }
 
diff --git a/src/core/qgsmaplayerlistutils.h b/src/core/qgsmaplayerlistutils.h
@@ -35,7 +35,7 @@ inline QList<QgsMapLayer *> _qgis_listQPointerToRaw( const QgsWeakMapLayerPointe
 {
   QList<QgsMapLayer *> lst;
   lst.reserve( layers.count() );
-  Q_FOREACH ( const QgsWeakMapLayerPointer &layerPtr, layers )
+  for ( const QgsWeakMapLayerPointer &layerPtr : layers )
   {
     if ( layerPtr )
       lst.append( layerPtr.data() );
@@ -47,7 +47,7 @@ inline QgsWeakMapLayerPointerList _qgis_listRawToQPointer( const QList<QgsMapLay
 {
   QgsWeakMapLayerPointerList lst;
   lst.reserve( layers.count() );
-  Q_FOREACH ( QgsMapLayer *layer, layers )
+  for ( QgsMapLayer *layer : layers )
   {
     lst.append( layer );
   }
@@ -58,7 +58,7 @@ inline QList<QgsMapLayer *> _qgis_listRefToRaw( const QList< QgsMapLayerRef > &l
 {
   QList<QgsMapLayer *> lst;
   lst.reserve( layers.count() );
-  Q_FOREACH ( const QgsMapLayerRef &layer, layers )
+  for ( const QgsMapLayerRef &layer : layers )
   {
     if ( layer )
       lst.append( layer.get() );
@@ -70,7 +70,7 @@ inline QList< QgsMapLayerRef > _qgis_listRawToRef( const QList<QgsMapLayer *> &l
 {
   QList< QgsMapLayerRef > lst;
   lst.reserve( layers.count() );
-  Q_FOREACH ( QgsMapLayer *layer, layers )
+  for ( QgsMapLayer *layer : layers )
   {
     lst.append( QgsMapLayerRef( layer ) );
   }
@@ -92,7 +92,7 @@ inline QStringList _qgis_listQPointerToIDs( const QgsWeakMapLayerPointerList &la
 {
   QStringList lst;
   lst.reserve( layers.count() );
-  Q_FOREACH ( const QgsWeakMapLayerPointer &layerPtr, layers )
+  for ( const QgsWeakMapLayerPointer &layerPtr : layers )
   {
     if ( layerPtr )
       lst << layerPtr->id();
@@ -107,7 +107,7 @@ inline static QgsMapLayer *_qgis_findLayer( const QList< QgsMapLayer *> &layers,
   QgsMapLayer *matchNameInsensitive = nullptr;
 
   // Look for match against layer IDs
-  Q_FOREACH ( QgsMapLayer *layer, layers )
+  for ( QgsMapLayer *layer : layers )
   {
     if ( !matchId && layer->id() == identifier )
     {
diff --git a/src/core/qgsmaplayerref.h b/src/core/qgsmaplayerref.h
@@ -171,7 +171,8 @@ struct _LayerRef
 
     if ( project && !name.isEmpty() )
     {
-      Q_FOREACH ( QgsMapLayer *l, project->mapLayersByName( name ) )
+      const QList<QgsMapLayer *> layers = project->mapLayersByName( name );
+      for ( QgsMapLayer *l : layers )
       {
         if ( TYPE *tl = qobject_cast< TYPE *>( l ) )
         {
diff --git a/src/core/qgsrelation.h b/src/core/qgsrelation.h
@@ -296,7 +296,7 @@ class CORE_EXPORT QgsRelation
     % MethodCode
     const QList< QgsRelation::FieldPair > &pairs = sipCpp->fieldPairs();
     sipRes = new QMap< QString, QString >();
-    Q_FOREACH ( const QgsRelation::FieldPair &pair, pairs )
+    for ( const QgsRelation::FieldPair &pair : pairs )
     {
       sipRes->insert( pair.first, pair.second );
     }
diff --git a/src/core/qgsrulebasedlabeling.cpp b/src/core/qgsrulebasedlabeling.cpp
@@ -86,6 +86,17 @@ void QgsRuleBasedLabeling::Rule::setSettings( QgsPalLayerSettings *settings )
   mSettings = settings;
 }
 
+QgsRuleBasedLabeling::RuleList QgsRuleBasedLabeling::Rule::descendants() const
+{
+  RuleList l;
+  for ( Rule *c : mChildren )
+  {
+    l += c;
+    l += c->descendants();
+  }
+  return l;
+}
+
 void QgsRuleBasedLabeling::Rule::initFilter()
 {
   if ( mElseRule || mFilterExp.compare( QLatin1String( "ELSE" ), Qt::CaseInsensitive ) == 0 )
diff --git a/src/core/qgsrulebasedlabeling.h b/src/core/qgsrulebasedlabeling.h
@@ -203,7 +203,7 @@ class CORE_EXPORT QgsRuleBasedLabeling : public QgsAbstractVectorLayerLabeling
          *
          * \returns A list of descendant rules
          */
-        QgsRuleBasedLabeling::RuleList descendants() const { RuleList l; Q_FOREACH ( Rule *c, mChildren ) { l += c; l += c->descendants(); } return l; }
+        QgsRuleBasedLabeling::RuleList descendants() const;
 
         /**
          * The parent rule
diff --git a/src/core/qgssqlstatement.cpp b/src/core/qgssqlstatement.cpp
@@ -251,6 +251,14 @@ bool QgsSQLStatement::doBasicValidationChecks( QString &errorMsgOut ) const
 ///////////////////////////////////////////////
 // nodes
 
+void QgsSQLStatement::NodeList::accept( QgsSQLStatement::Visitor &v ) const
+{
+  for ( QgsSQLStatement::Node *node : mList )
+  {
+    node->accept( v );
+  }
+}
+
 QgsSQLStatement::NodeList *QgsSQLStatement::NodeList::clone() const
 {
   NodeList *nl = new NodeList;
diff --git a/src/core/qgssqlstatement.h b/src/core/qgssqlstatement.h
@@ -301,7 +301,7 @@ class CORE_EXPORT QgsSQLStatement
         int count() const { return mList.count(); }
 
         //! Accept visitor
-        void accept( QgsSQLStatement::Visitor &v ) const { Q_FOREACH ( QgsSQLStatement::Node *node, mList ) { node->accept( v ); } }
+        void accept( QgsSQLStatement::Visitor &v ) const;
 
         //! Creates a deep copy of this list. Ownership is transferred to the caller
         QgsSQLStatement::NodeList *clone() const SIP_FACTORY;
diff --git a/src/core/symbology/qgscolorbrewerpalette.cpp b/src/core/symbology/qgscolorbrewerpalette.cpp
@@ -291,3 +291,56 @@ const char *QgsColorBrewerPalette::BREWER_STRING =
   "PuBuGn-7-246,239,247 208,209,230 166,189,219 103,169,207 54,144,192 2,129,138 1,100,80\n"
   "PuBuGn-8-255,247,251 236,226,240 208,209,230 166,189,219 103,169,207 54,144,192 2,129,138 1,100,80\n"
   "PuBuGn-9-255,247,251 236,226,240 208,209,230 166,189,219 103,169,207 54,144,192 2,129,138 1,108,89 1,70,54";
+
+QList<QColor> QgsColorBrewerPalette::listSchemeColors( const QString &schemeName, int colors )
+{
+  QList<QColor> pal;
+  QString palette( BREWER_STRING );
+  const QStringList list = palette.split( QChar( '\n' ) );
+  for ( const QString &entry : list )
+  {
+    QStringList items = entry.split( QChar( '-' ) );
+    if ( items.count() != 3 || items[0] != schemeName || items[1].toInt() != colors )
+      continue;
+    const QStringList colors = items[2].split( QChar( ' ' ) );
+    for ( const QString &clr : colors )
+    {
+      pal << QgsSymbolLayerUtils::parseColor( clr );
+    }
+  }
+  return pal;
+}
+
+QStringList QgsColorBrewerPalette::listSchemes()
+{
+  QStringList schemes;
+
+  QString palette( BREWER_STRING );
+  const QStringList list = palette.split( QChar( '\n' ) );
+  for ( const QString &entry : list )
+  {
+    QStringList items = entry.split( QChar( '-' ) );
+    if ( items.count() != 3 )
+      continue;
+    if ( !schemes.contains( items[0] ) )
+      schemes << items[0];
+  }
+  return schemes;
+}
+
+QList<int> QgsColorBrewerPalette::listSchemeVariants( const QString &schemeName )
+{
+  QList<int> variants;
+
+  QString palette( BREWER_STRING );
+  const QStringList list = palette.split( QChar( '\n' ) );
+  for ( const QString &entry : list )
+  {
+    QStringList items = entry.split( QChar( '-' ) );
+    if ( items.count() != 3 || items[0] != schemeName )
+      continue;
+    variants << items[1].toInt();
+  }
+
+  return variants;
+}
diff --git a/src/core/symbology/qgscolorbrewerpalette.h b/src/core/symbology/qgscolorbrewerpalette.h
@@ -28,58 +28,11 @@
 class CORE_EXPORT QgsColorBrewerPalette
 {
   public:
-    static QList<QColor> listSchemeColors( const QString &schemeName, int colors )
-    {
-      QList<QColor> pal;
-      QString palette( BREWER_STRING );
-      QStringList list = palette.split( QChar( '\n' ) );
-      Q_FOREACH ( const QString &entry, list )
-      {
-        QStringList items = entry.split( QChar( '-' ) );
-        if ( items.count() != 3 || items[0] != schemeName || items[1].toInt() != colors )
-          continue;
-        QStringList colors = items[2].split( QChar( ' ' ) );
-        Q_FOREACH ( const QString &clr, colors )
-        {
-          pal << QgsSymbolLayerUtils::parseColor( clr );
-        }
-      }
-      return pal;
-    }
+    static QList<QColor> listSchemeColors( const QString &schemeName, int colors );
 
-    static QStringList listSchemes()
-    {
-      QStringList schemes;
+    static QStringList listSchemes();
 
-      QString palette( BREWER_STRING );
-      QStringList list = palette.split( QChar( '\n' ) );
-      Q_FOREACH ( const QString &entry, list )
-      {
-        QStringList items = entry.split( QChar( '-' ) );
-        if ( items.count() != 3 )
-          continue;
-        if ( !schemes.contains( items[0] ) )
-          schemes << items[0];
-      }
-      return schemes;
-    }
-
-    static QList<int> listSchemeVariants( const QString &schemeName )
-    {
-      QList<int> variants;
-
-      QString palette( BREWER_STRING );
-      QStringList list = palette.split( QChar( '\n' ) );
-      Q_FOREACH ( const QString &entry, list )
-      {
-        QStringList items = entry.split( QChar( '-' ) );
-        if ( items.count() != 3 || items[0] != schemeName )
-          continue;
-        variants << items[1].toInt();
-      }
-
-      return variants;
-    }
+    static QList<int> listSchemeVariants( const QString &schemeName );
 
     // extracted ColorBrewer data
     static const char *BREWER_STRING;
diff --git a/src/core/symbology/qgsrulebasedrenderer.cpp b/src/core/symbology/qgsrulebasedrenderer.cpp
@@ -701,6 +701,17 @@ QgsRuleBasedRenderer::Rule *QgsRuleBasedRenderer::Rule::create( QDomElement &rul
   return rule;
 }
 
+QgsRuleBasedRenderer::RuleList QgsRuleBasedRenderer::Rule::descendants() const
+{
+  RuleList l;
+  for ( QgsRuleBasedRenderer::Rule *c : mChildren )
+  {
+    l += c;
+    l += c->descendants();
+  }
+  return l;
+}
+
 QgsRuleBasedRenderer::Rule *QgsRuleBasedRenderer::Rule::createFromSld( QDomElement &ruleElem, QgsWkbTypes::GeometryType geomType )
 {
   if ( ruleElem.localName() != QLatin1String( "Rule" ) )
diff --git a/src/core/symbology/qgsrulebasedrenderer.h b/src/core/symbology/qgsrulebasedrenderer.h
@@ -84,7 +84,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
     struct RenderLevel
     {
       explicit RenderLevel( int z ): zIndex( z ) {}
-      ~RenderLevel() { Q_FOREACH ( RenderJob *j, jobs ) delete j; }
+      ~RenderLevel() { qDeleteAll( jobs ); }
       int zIndex;
 
       //! List of jobs to render, owned by this object.
@@ -95,7 +95,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
         zIndex = rh.zIndex;
         qDeleteAll( jobs );
         jobs.clear();
-        Q_FOREACH ( RenderJob *job, rh.jobs )
+        for ( RenderJob *job :  qgis::as_const( rh.jobs ) )
         {
           jobs << new RenderJob( *job );
         }
@@ -105,7 +105,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
       RenderLevel( const QgsRuleBasedRenderer::RenderLevel &other )
         : zIndex( other.zIndex )
       {
-        Q_FOREACH ( RenderJob *job, other.jobs )
+        for ( RenderJob *job : qgis::as_const( other.jobs ) )
         {
           jobs << new RenderJob( *job );
         }
@@ -381,7 +381,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
          *
          * \returns A list of descendant rules
          */
-        QgsRuleBasedRenderer::RuleList descendants() const { RuleList l; Q_FOREACH ( QgsRuleBasedRenderer::Rule *c, mChildren ) { l += c; l += c->descendants(); } return l; }
+        QgsRuleBasedRenderer::RuleList descendants() const;
 
         /**
          * The parent rule
diff --git a/src/gui/raster/qgspalettedrendererwidget.cpp b/src/gui/raster/qgspalettedrendererwidget.cpp
@@ -803,5 +803,40 @@ void QgsPalettedRendererModel::deleteAll()
   emit classesChanged();
 }
 
-///@endcond PRIVATE
+void QgsPalettedRendererClassGatherer::run()
+{
+  mWasCanceled = false;
+
+  // allow responsive cancelation
+  mFeedback = new QgsRasterBlockFeedback();
+  connect( mFeedback, &QgsRasterBlockFeedback::progressChanged, this, &QgsPalettedRendererClassGatherer::progressChanged );
+
+  QgsPalettedRasterRenderer::ClassData newClasses = QgsPalettedRasterRenderer::classDataFromRaster( mLayer->dataProvider(), mBandNumber, mRamp.get(), mFeedback );
 
+  // combine existing classes with new classes
+  QgsPalettedRasterRenderer::ClassData::iterator classIt = newClasses.begin();
+  for ( ; classIt != newClasses.end(); ++classIt )
+  {
+    // check if existing classes contains this same class
+    for ( const QgsPalettedRasterRenderer::Class &existingClass : qgis::as_const( mClasses ) )
+    {
+      if ( existingClass.value == classIt->value )
+      {
+        classIt->color = existingClass.color;
+        classIt->label = existingClass.label;
+        break;
+      }
+    }
+  }
+  mClasses = newClasses;
+
+  // be overly cautious - it's *possible* stop() might be called between deleting mFeedback and nulling it
+  mFeedbackMutex.lock();
+  delete mFeedback;
+  mFeedback = nullptr;
+  mFeedbackMutex.unlock();
+
+  emit collectedClasses();
+}
+
+///@endcond PRIVATE
diff --git a/src/gui/raster/qgspalettedrendererwidget.h b/src/gui/raster/qgspalettedrendererwidget.h

Original file line number	Diff line number	Diff line change
`@@ -238,7 +238,7 @@ The second element of each pair are the field names of the matching primary key.`
`238`	`238`	`%MethodCode`
`239`	`239`	`const QList< QgsRelation::FieldPair > &pairs = sipCpp->fieldPairs();`
`240`	`240`	`sipRes = new QMap< QString, QString >();`
`241`		`- Q_FOREACH ( const QgsRelation::FieldPair &pair, pairs )`
	`241`	`+ for ( const QgsRelation::FieldPair &pair : pairs )`
`242`	`242`	`{`
`243`	`243`	`sipRes->insert( pair.first, pair.second );`
`244`	`244`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ class QgsExpressionSorter`
`36`	`36`	`bool operator()( const QgsIndexedFeature &f1, const QgsIndexedFeature &f2 ) const`
`37`	`37`	`{`
`38`	`38`	`int i = 0;`
`39`		`- Q_FOREACH ( const QgsFeatureRequest::OrderByClause &orderBy, mPreparedOrderBys )`
	`39`	`+ for ( const QgsFeatureRequest::OrderByClause &orderBy : qgis::as_const( mPreparedOrderBys ) )`
`40`	`40`	`{`
`41`	`41`	`const QVariant &v1 = f1.mIndexes.at( i );`
`42`	`42`	`const QVariant &v2 = f2.mIndexes.at( i );`
`@@ -143,15 +143,15 @@ class QgsExpressionSorter`
`143`	`143`
`144`	`144`	`QgsIndexedFeature indexedFeature;`
`145`	`145`
`146`		`- Q_FOREACH ( const QgsFeature &f, features )`
	`146`	`+ for ( const QgsFeature &f : qgis::as_const( features ) )`
`147`	`147`	`{`
`148`	`148`	`indexedFeature.mIndexes.resize( mPreparedOrderBys.size() );`
`149`	`149`	`indexedFeature.mFeature = f;`
`150`	`150`
`151`	`151`	`expressionContext->setFeature( indexedFeature.mFeature );`
`152`	`152`
`153`	`153`	`int i = 0;`
`154`		`- Q_FOREACH ( const QgsFeatureRequest::OrderByClause &orderBy, mPreparedOrderBys )`
	`154`	`+ for ( const QgsFeatureRequest::OrderByClause &orderBy : qgis::as_const( mPreparedOrderBys ) )`
`155`	`155`	`{`
`156`	`156`	`indexedFeature.mIndexes.replace( i++, orderBy.expression().evaluate( expressionContext ) );`
`157`	`157`	`}`
`@@ -164,7 +164,7 @@ class QgsExpressionSorter`
`164`	`164`
`165`	`165`	`features.clear();`
`166`	`166`
`167`		`- Q_FOREACH ( const QgsIndexedFeature &indexedFeature, indexedFeatures )`
	`167`	`+ for ( const QgsIndexedFeature &indexedFeature : qgis::as_const( indexedFeatures ) )`
`168`	`168`	`features.append( indexedFeature.mFeature );`
`169`	`169`	`}`
`170`	`170`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ inline QList<QgsMapLayer *> _qgis_listQPointerToRaw( const QgsWeakMapLayerPointe`
`35`	`35`	`{`
`36`	`36`	`QList<QgsMapLayer *> lst;`
`37`	`37`	`lst.reserve( layers.count() );`
`38`		`- Q_FOREACH ( const QgsWeakMapLayerPointer &layerPtr, layers )`
	`38`	`+ for ( const QgsWeakMapLayerPointer &layerPtr : layers )`
`39`	`39`	`{`
`40`	`40`	`if ( layerPtr )`
`41`	`41`	`lst.append( layerPtr.data() );`
`@@ -47,7 +47,7 @@ inline QgsWeakMapLayerPointerList _qgis_listRawToQPointer( const QList<QgsMapLay`
`47`	`47`	`{`
`48`	`48`	`QgsWeakMapLayerPointerList lst;`
`49`	`49`	`lst.reserve( layers.count() );`
`50`		`- Q_FOREACH ( QgsMapLayer *layer, layers )`
	`50`	`+ for ( QgsMapLayer *layer : layers )`
`51`	`51`	`{`
`52`	`52`	`lst.append( layer );`
`53`	`53`	`}`
`@@ -58,7 +58,7 @@ inline QList<QgsMapLayer *> _qgis_listRefToRaw( const QList< QgsMapLayerRef > &l`
`58`	`58`	`{`
`59`	`59`	`QList<QgsMapLayer *> lst;`
`60`	`60`	`lst.reserve( layers.count() );`
`61`		`- Q_FOREACH ( const QgsMapLayerRef &layer, layers )`
	`61`	`+ for ( const QgsMapLayerRef &layer : layers )`
`62`	`62`	`{`
`63`	`63`	`if ( layer )`
`64`	`64`	`lst.append( layer.get() );`
`@@ -70,7 +70,7 @@ inline QList< QgsMapLayerRef > _qgis_listRawToRef( const QList<QgsMapLayer *> &l`
`70`	`70`	`{`
`71`	`71`	`QList< QgsMapLayerRef > lst;`
`72`	`72`	`lst.reserve( layers.count() );`
`73`		`- Q_FOREACH ( QgsMapLayer *layer, layers )`
	`73`	`+ for ( QgsMapLayer *layer : layers )`
`74`	`74`	`{`
`75`	`75`	`lst.append( QgsMapLayerRef( layer ) );`
`76`	`76`	`}`
`@@ -92,7 +92,7 @@ inline QStringList _qgis_listQPointerToIDs( const QgsWeakMapLayerPointerList &la`
`92`	`92`	`{`
`93`	`93`	`QStringList lst;`
`94`	`94`	`lst.reserve( layers.count() );`
`95`		`- Q_FOREACH ( const QgsWeakMapLayerPointer &layerPtr, layers )`
	`95`	`+ for ( const QgsWeakMapLayerPointer &layerPtr : layers )`
`96`	`96`	`{`
`97`	`97`	`if ( layerPtr )`
`98`	`98`	`lst << layerPtr->id();`
`@@ -107,7 +107,7 @@ inline static QgsMapLayer _qgis_findLayer( const QList< QgsMapLayer > &layers,`
`107`	`107`	`QgsMapLayer *matchNameInsensitive = nullptr;`
`108`	`108`
`109`	`109`	`// Look for match against layer IDs`
`110`		`- Q_FOREACH ( QgsMapLayer *layer, layers )`
	`110`	`+ for ( QgsMapLayer *layer : layers )`
`111`	`111`	`{`
`112`	`112`	`if ( !matchId && layer->id() == identifier )`
`113`	`113`	`{`
Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,8 @@ struct _LayerRef`
`171`	`171`
`172`	`172`	`if ( project && !name.isEmpty() )`
`173`	`173`	`{`
`174`		`- Q_FOREACH ( QgsMapLayer *l, project->mapLayersByName( name ) )`
	`174`	`+ const QList<QgsMapLayer *> layers = project->mapLayersByName( name );`
	`175`	`+ for ( QgsMapLayer *l : layers )`
`175`	`176`	`{`
`176`	`177`	`if ( TYPE tl = qobject_cast< TYPE >( l ) )`
`177`	`178`	`{`
Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,7 @@ class CORE_EXPORT QgsRelation`
`296`	`296`	`% MethodCode`
`297`	`297`	`const QList< QgsRelation::FieldPair > &pairs = sipCpp->fieldPairs();`
`298`	`298`	`sipRes = new QMap< QString, QString >();`
`299`		`- Q_FOREACH ( const QgsRelation::FieldPair &pair, pairs )`
	`299`	`+ for ( const QgsRelation::FieldPair &pair : pairs )`
`300`	`300`	`{`
`301`	`301`	`sipRes->insert( pair.first, pair.second );`
`302`	`302`	`}`
Original file line number	Diff line number	Diff line change
`@@ -203,7 +203,7 @@ class CORE_EXPORT QgsRuleBasedLabeling : public QgsAbstractVectorLayerLabeling`
`203`	`203`	`*`
`204`	`204`	`* \returns A list of descendant rules`
`205`	`205`	`*/`
`206`		`- QgsRuleBasedLabeling::RuleList descendants() const { RuleList l; Q_FOREACH ( Rule *c, mChildren ) { l += c; l += c->descendants(); } return l; }`
	`206`	`+ QgsRuleBasedLabeling::RuleList descendants() const;`
`207`	`207`
`208`	`208`	`/**`
`209`	`209`	`* The parent rule`