Changeset 2633 in josm for trunk

Timestamp:

2009-12-14T15:30:24+01:00 (15 years ago)

Author:

mjulius

Message:

small optimization - use DataSet.getSelectedWays() instead of iterating over all primitives

File:

• trunk/src/org/openstreetmap/josm/actions/ReverseWayAction.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/actions/ReverseWayAction.java

@@ -2 +2 @@
 package org.openstreetmap.josm.actions;
 
+import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
 import static org.openstreetmap.josm.tools.I18n.tr;
-import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
 
 import java.awt.event.ActionEvent;
@@ -20 +20 @@
 import org.openstreetmap.josm.corrector.ReverseWayTagCorrector;
 import org.openstreetmap.josm.corrector.UserCancelException;
-import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.Node;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
@@ -40 +39 @@
             return;
 
-        final Collection<Way> sel = new LinkedList<Way>();
-        for (OsmPrimitive primitive : getCurrentDataSet().getSelected()) {
-            if (primitive instanceof Way) {
-                sel.add((Way)primitive);
-            }
-        }
+        final Collection<Way> sel = getCurrentDataSet().getSelectedWays();
         if (sel.isEmpty()) {
             JOptionPane.showMessageDialog(