Changeset 8856 in josm for trunk/src/org/openstreetmap/josm/io/imagery

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/io/imagery/ImageryReader.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

@@ -4 +4 @@
 import java.io.IOException;
 import java.io.InputStream;
-import java.util.ArrayDeque;
 import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.Deque;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.Stack;
 
 import javax.xml.parsers.ParserConfigurationException;
@@ -73 +72 @@
         private StringBuilder accumulator = new StringBuilder();
 
-        private Deque<State> states;
+        private Stack<State> states;
 
         private List<ImageryInfo> entries;
@@ -96 +95 @@
             accumulator = new StringBuilder();
             skipEntry = false;
-            states = new ArrayDeque<>();
+            states = new Stack<>();
             states.push(State.INIT);
             entries = new ArrayList<>();