Changeset 18818 in josm for trunk/src/org/openstreetmap/josm/io/GpxParser.java

Timestamp:

2023-08-23T00:10:21+02:00 (15 months ago)

Author:

taylor.smock

Message:

Fix tests

File:

• trunk/src/org/openstreetmap/josm/io/GpxParser.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/io/GpxParser.java

@@ -18 +18 @@
 
 import java.time.DateTimeException;
-import java.util.ArrayDeque;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Deque;
 import java.util.HashMap;
 import java.util.LinkedList;
@@ -27 +25 @@
 import java.util.Map;
 import java.util.Optional;
+import java.util.Stack;
 
 import org.openstreetmap.josm.data.Bounds;
@@ -77 +76 @@
     private GpxExtensionCollection currentExtensionCollection;
     private GpxExtensionCollection currentTrackExtensionCollection;
-    private Deque<State> states;
-    private final Deque<String[]> elements = new ArrayDeque<>();
+    private final Stack<State> states = new Stack<>();
+    private final Stack<String[]> elements = new Stack<>();
 
     private StringBuilder accumulator = new StringBuilder();
@@ -87 +86 @@
     public void startDocument() {
         accumulator = new StringBuilder();
-        states = new ArrayDeque<>();
         data = new GpxData(true);
         currentExtensionCollection = new GpxExtensionCollection();
@@ -402 +400 @@
      */
     private void startElementExt(String namespaceURI, String qName, Attributes attributes) {
-        if (states.peekLast() == State.TRK) {
+        if (states.lastElement() == State.TRK) {
             currentTrackExtensionCollection.openChild(namespaceURI, qName, attributes);
         } else {
@@ -651 +649 @@
                 convertUrlToLink(currentWayPoint.attr);
                 currentWayPoint.getExtensions().addAll(currentExtensionCollection);
-                if (!currentWayPoint.isLatLonKnown()) {
-                    currentExtensionCollection.clear();
-                    throw new SAXException(tr("{0} element does not have valid latitude and/or longitude.", "wpt"));
-                }
                 data.waypoints.add(currentWayPoint);
                 currentExtensionCollection.clear();
@@ -723 +717 @@
         } else if (currentExtensionCollection != null) {
             String acc = accumulator.toString().trim();
-            if (states.peekLast() == State.TRK) {
+            if (states.lastElement() == State.TRK) {
                 currentTrackExtensionCollection.closeChild(qName, acc); //a segment inside the track can have an extension too
             } else {