Changeset 8856 in josm for trunk/src/org/openstreetmap/josm/actions/UploadSelectionAction.java

Timestamp:

2015-10-11T15:28:33+02:00 (9 years ago)

Author:

Don-vip

Message:

fix #11957 - partial revert of r8851 - do not replace Stack by ArrayDeque because of different iteration behaviour + add unit test

File:

• trunk/src/org/openstreetmap/josm/actions/UploadSelectionAction.java (modified) (2 diffs)

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

@@ -8 +8 @@
 import java.awt.event.KeyEvent;
 import java.io.IOException;
-import java.util.ArrayDeque;
 import java.util.Collection;
-import java.util.Deque;
 import java.util.HashSet;
 import java.util.Set;
+import java.util.Stack;
 
 import javax.swing.JOptionPane;
@@ -322 +321 @@
         protected void realRun() throws SAXException, IOException, OsmTransferException {
             try {
-                Deque<OsmPrimitive> toCheck = new ArrayDeque<>();
+                Stack<OsmPrimitive> toCheck = new Stack<>();
                 toCheck.addAll(getPrimitivesToCheckForParents());
                 Set<OsmPrimitive> checked = new HashSet<>();