Remove foreach, initialize pointer

qgis · Jun 11, 2018 · 404f07d · 404f07d
1 parent 0a325f1
commit 404f07d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/app/qgslayertreeviewfilterindicator.cpp b/src/app/qgslayertreeviewfilterindicator.cpp
@@ -175,7 +175,7 @@ void QgsLayerTreeViewFilterIndicatorProvider::addOrRemoveIndicator( QgsLayerTree
     const QList<QgsLayerTreeViewIndicator *> nodeIndicators = mLayerTreeView->indicators( node );
 
     // maybe the indicator exists already
-    foreach ( QgsLayerTreeViewIndicator *indicator, nodeIndicators )
+    for ( QgsLayerTreeViewIndicator *indicator : nodeIndicators )
     {
       if ( mIndicators.contains( indicator ) )
       {
@@ -192,7 +192,7 @@ void QgsLayerTreeViewFilterIndicatorProvider::addOrRemoveIndicator( QgsLayerTree
     const QList<QgsLayerTreeViewIndicator *> nodeIndicators = mLayerTreeView->indicators( node );
 
     // there may be existing indicator we need to get rid of
-    foreach ( QgsLayerTreeViewIndicator *indicator, nodeIndicators )
+    for ( QgsLayerTreeViewIndicator *indicator : nodeIndicators )
     {
       if ( mIndicators.contains( indicator ) )
       {

diff --git a/src/app/qgslayertreeviewfilterindicator.h b/src/app/qgslayertreeviewfilterindicator.h
@@ -50,7 +50,7 @@ class QgsLayerTreeViewFilterIndicatorProvider : public QObject
     void addOrRemoveIndicator( QgsLayerTreeNode *node, QgsVectorLayer *vlayer );
 
   private:
-    QgsLayerTreeView *mLayerTreeView;
+    QgsLayerTreeView *mLayerTreeView = nullptr;
     QIcon mIcon;
     QSet<QgsLayerTreeViewIndicator *> mIndicators;
 };