Changeset 8332 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint/Cascade.java

Timestamp:

2015-05-06T00:23:51+02:00 (11 years ago)

Author:

Don-vip

Message:

fix various issues:

• performance - inefficient use of keySet iterator instead of entrySet iterator
• squid:S2674: The value returned from a stream read should be checked
• remove unused imports
• improve javadoc

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/Cascade.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/Cascade.java

@@ -7 +7 @@
 import java.util.List;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.regex.Pattern;
 
@@ -204 +205 @@
     public String toString() {
         StringBuilder res = new StringBuilder("Cascade{ ");
-        for (String key : prop.keySet()) {
-            res.append(key+":");
-            Object val = prop.get(key);
+        for (Entry<String, Object> entry : prop.entrySet()) {
+            res.append(entry.getKey()+":");
+            Object val = entry.getValue();
             if (val instanceof float[]) {
                 res.append(Arrays.toString((float[]) val));