Avoid iterating over temporary container

qgis · May 13, 2016 · 63cab56839d6795451223150d8102e4e056ab8b1 · 63cab56
1 parent a3d6227
commit 63cab56839d6795451223150d8102e4e056ab8b1
Unified Split

Showing with 3 additions and 2 deletions.

+3 −2 src/plugins/gps_importer/qgsgpsplugin.cpp
diff --git a/src/plugins/gps_importer/qgsgpsplugin.cpp b/src/plugins/gps_importer/qgsgpsplugin.cpp
@@ -124,8 +124,9 @@ void QgsGPSPlugin::run()
   std::vector<QgsVectorLayer*> gpxLayers;
   QMap<QString, QgsMapLayer*>::const_iterator iter;
   QgsMapLayerRegistry* registry = QgsMapLayerRegistry::instance();
-  for ( iter =  registry->mapLayers().begin();
-        iter != registry->mapLayers().end(); ++iter )
+  QMap<QString, QgsMapLayer*> layers = registry->mapLayers();
+  for ( iter = layers.constBegin();
+        iter != layers.constEnd(); ++iter )
   {
     if ( iter.value()->type() == QgsMapLayer::VectorLayer )
     {