Changeset 6316 in josm for trunk/src/org/openstreetmap/josm/data/imagery

Timestamp:

2013-10-07T20:18:17+02:00 (11 years ago)

Author:

Don-vip

Message:

Sonar/FindBugs - Loose coupling

Location:

trunk/src/org/openstreetmap/josm/data/imagery

Files:

• ImageryLayerInfo.java (modified) (2 diffs)
• OffsetBookmark.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java

@@ -22 +22 @@
 
     public static final ImageryLayerInfo instance = new ImageryLayerInfo();
-    ArrayList<ImageryInfo> layers = new ArrayList<ImageryInfo>();
-    static ArrayList<ImageryInfo> defaultLayers = new ArrayList<ImageryInfo>();
+    List<ImageryInfo> layers = new ArrayList<ImageryInfo>();
+    static List<ImageryInfo> defaultLayers = new ArrayList<ImageryInfo>();
 
     private final static String[] DEFAULT_LAYER_SITES = {
@@ -92 +92 @@
 
         Collection<String> defaults = Main.pref.getCollection("imagery.layers.default");
-        ArrayList<String> defaultsSave = new ArrayList<String>();
+        List<String> defaultsSave = new ArrayList<String>();
         for (ImageryInfo def : defaultLayers) {
             if (def.isDefaultEntry()) {

trunk/src/org/openstreetmap/josm/data/imagery/OffsetBookmark.java

@@ -45 +45 @@
 
     public OffsetBookmark(Collection<String> list) {
-        ArrayList<String> array = new ArrayList<String>(list);
+        List<String> array = new ArrayList<String>(list);
         this.projectionCode = array.get(0);
         this.layerName = array.get(1);
@@ -60 +60 @@
     }
 
-    public ArrayList<String> getInfoArray() {
-        ArrayList<String> res = new ArrayList<String>(7);
+    public List<String> getInfoArray() {
+        List<String> res = new ArrayList<String>(7);
         if (projectionCode != null) {
             res.add(projectionCode);