Changeset 11891 in josm for trunk/src/org/openstreetmap/josm/tools/ImageWarp.java

Timestamp:

2017-04-13T00:03:15+02:00 (9 years ago)

Author:

Don-vip

Message:

see #7427 - sonar - squid:S1226 - Method parameters, caught exceptions and foreach variables should not be reassigned

File:

• trunk/src/org/openstreetmap/josm/tools/ImageWarp.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/tools/ImageWarp.java ¶

@@ -119 +119 @@
     }
 
+    private static int clamp(int i, int max) {
+        if (i < 0) {
+            return 0;
+        } else if (i >= max) {
+            return max - 1;
+        } else {
+            return i;
+        }
+    }
+
     private static Color getColor(int x, int y, BufferedImage img) {
         // border strategy: continue with the color of the outermost pixel,
         // but change alpha component to fully translucent
-        boolean transparent = false;
-        if (x < 0) {
-            x = 0;
-            transparent = true;
-        } else if (x >= img.getWidth()) {
-            x = img.getWidth() - 1;
-            transparent = true;
-        }
-        if (y < 0) {
-            y = 0;
-            transparent = true;
-        } else if (y >= img.getHeight()) {
-            y = img.getHeight() - 1;
-            transparent = true;
-        }
-        Color clr = new Color(img.getRGB(x, y));
-        if (!transparent)
+        int a = clamp(x, img.getWidth());
+        int b = clamp(y, img.getHeight());
+        Color clr = new Color(img.getRGB(a, b));
+        if (a == x && b == y)
             return clr;
         // keep color components, but set transparency to 0