Changeset 8851 in josm for trunk/src/org/openstreetmap/josm/actions/upload/CyclicUploadDependencyException.java

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/actions/upload/CyclicUploadDependencyException.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/actions/upload/CyclicUploadDependencyException.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Stack;
+import java.util.ArrayDeque;
+import java.util.Deque;
 
 import org.openstreetmap.josm.data.osm.Relation;
 
 public class CyclicUploadDependencyException extends Exception {
-    private final Stack<Relation> cycle;
+    private final Deque<Relation> cycle;
 
-    public CyclicUploadDependencyException(Stack<Relation> cycle) {
+    public CyclicUploadDependencyException(Deque<Relation> cycle) {
         this.cycle = cycle;
     }
@@ -34 +33 @@
         sb.append(tr("Cyclic dependency between relations:"))
           .append('[');
-        for (int i = 0; i < cycle.size(); i++) {
-            if (i > 0) {
+        for (Relation r : cycle) {
+            if (sb.length() > 0) {
                 sb.append(',');
             }
-            sb.append(formatRelation(cycle.get(i)));
+            sb.append(formatRelation(r));
         }
         sb.append(']');
@@ -44 +43 @@
     }
 
-    public List<Relation> getCyclicUploadDependency() {
-        return new ArrayList<>(cycle);
+    public Deque<Relation> getCyclicUploadDependency() {
+        return new ArrayDeque<>(cycle);
     }
 }