Changeset 9999 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ParentRelationLoadingTask.java

Timestamp:

2016-03-15T09:15:57+01:00 (8 years ago)

Author:

Don-vip

Message:

sonar - do not copy collection contents from one to another with a loop

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ParentRelationLoadingTask.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ParentRelationLoadingTask.java

@@ -12 +12 @@
 
 import org.openstreetmap.josm.Main;
-import org.openstreetmap.josm.data.DataSource;
 import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.DataSetMerger;
@@ -165 +164 @@
 
                 // copy the merged layer's data source info
-                for (DataSource src : referrers.dataSources) {
-                    getLayer().data.dataSources.add(src);
-                }
+                getLayer().data.dataSources.addAll(referrers.dataSources);
                 // FIXME: this is necessary because there are  dialogs listening
-                // for DataChangeEvents which manipulate Swing components on this
-                // thread.
+                // for DataChangeEvents which manipulate Swing components on this thread.
                 //
                 SwingUtilities.invokeLater(