Changeset 17093 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2020-10-07T19:44:51+02:00 (4 years ago)

Author:

simon04

Message:

Use Collection.addAll

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• conflict/tags/TagConflictResolutionUtil.java (modified) (1 diff)
• dialogs/ConflictDialog.java (modified) (1 diff)
• dialogs/changeset/ChangesetCacheManagerModel.java (modified) (1 diff)
• layer/AutosaveTask.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/conflict/tags/TagConflictResolutionUtil.java

@@ -299 +299 @@
             Set<String> results = instantiateSortedSet();
             for (String value: values) {
-                for (String part: value.split(Pattern.quote(separator), -1)) {
-                    results.add(part);
-                }
+                String[] parts = value.split(Pattern.quote(separator), -1);
+                results.addAll(Arrays.asList(parts));
             }
             return String.join(separator, results);

trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictDialog.java

@@ -483 +483 @@
             Collection<OsmPrimitive> sel = new LinkedList<>();
             synchronized (this) {
-                for (OsmPrimitive o : lstConflicts.getSelectedValuesList()) {
-                    sel.add(o);
-                }
+                sel.addAll(lstConflicts.getSelectedValuesList());
             }
             DataSet ds = MainApplication.getLayerManager().getEditDataSet();

trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManagerModel.java

@@ -172 +172 @@
     public void changesetCacheUpdated(ChangesetCacheEvent event) {
         List<Changeset> selected = getSelectedChangesets();
-        for (Changeset cs: event.getAddedChangesets()) {
-            data.add(cs);
-        }
-        for (Changeset cs: event.getRemovedChangesets()) {
-            data.remove(cs);
-        }
+        data.addAll(event.getAddedChangesets());
+        data.removeAll(event.getRemovedChangesets());
         for (Changeset cs: event.getUpdatedChangesets()) {
             int idx = data.indexOf(cs);

trunk/src/org/openstreetmap/josm/gui/layer/AutosaveTask.java

@@ -15 +15 @@
 import java.nio.file.Path;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.Date;
@@ -149 +150 @@
             File[] files = deletedLayersDir.listFiles();
             if (files != null) {
-                for (File f: files) {
-                    deletedLayers.add(f); // FIXME: sort by mtime
-                }
+                // FIXME: sort by mtime
+                deletedLayers.addAll(Arrays.asList(files));
             }