Changeset 8851 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2015-10-10T15:47:41+02:00 (9 years ago)

Author:

Don-vip

Message:

sonar - squid:S1149 - Synchronized classes Vector, Hashtable, Stack and StringBuffer should not be used

File:

• trunk/src/org/openstreetmap/josm/data/APIDataSet.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/data/APIDataSet.java

@@ -2 +2 @@
 package org.openstreetmap.josm.data;
 
+import java.util.ArrayDeque;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Comparator;
+import java.util.Deque;
 import java.util.HashMap;
 import java.util.HashSet;
@@ -12 +14 @@
 import java.util.Map;
 import java.util.Set;
-import java.util.Stack;
 
 import org.openstreetmap.josm.actions.upload.CyclicUploadDependencyException;
@@ -309 +310 @@
         }
 
-        protected void visit(Stack<Relation> path, Relation current) throws CyclicUploadDependencyException {
+        protected void visit(Deque<Relation> path, Relation current) throws CyclicUploadDependencyException {
             if (path.contains(current)) {
                 path.push(current);
@@ -328 +329 @@
             visited = new HashSet<>();
             uploadOrder = new LinkedList<>();
-            Stack<Relation> path = new Stack<>();
+            Deque<Relation> path = new ArrayDeque<>();
             for (Relation relation: relations) {
                 visit(path, relation);