Changeset 13129 in josm for trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java

Timestamp:

2017-11-19T19:07:47+01:00 (6 years ago)

Author:

Don-vip

Message:

see #15476 - fix checkstyle/pmd warnings, add some javadoc

File:

• trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java

@@ -67 +67 @@
     private static final BooleanProperty AGPIFO_STYLE2 =
         new BooleanProperty("geoimage.agpifo-style-drag-and-zoom", false);
-    private static int DRAG_BUTTON;
-    private static int ZOOM_BUTTON;
+    private static int dragButton;
+    private static int zoomButton;
 
     /** Alternative to mouse wheel zoom; esp. handy if no mouse wheel is present **/
@@ -87 +87 @@
     private static final BooleanProperty BILIN_UPSAMP =
         new BooleanProperty("geoimage.bilinear-upsampling", false);
-    private static double BILIN_UPPER;
-    private static double BILIN_LOWER;
+    private static double bilinUpper;
+    private static double bilinLower;
 
     @Override
     public void preferenceChanged(PreferenceChangeEvent e) {
         if (e == null ||
-            e.getKey().equals(AGPIFO_STYLE2.getKey()))
-        {
-            DRAG_BUTTON = AGPIFO_STYLE2.get() ? 1 : 3;
-            ZOOM_BUTTON = DRAG_BUTTON == 1 ? 3 : 1;
+            e.getKey().equals(AGPIFO_STYLE2.getKey())) {
+            dragButton = AGPIFO_STYLE2.get() ? 1 : 3;
+            zoomButton = dragButton == 1 ? 3 : 1;
         }
         if (e == null ||
             e.getKey().equals(MAX_ZOOM.getKey()) ||
             e.getKey().equals(BILIN_DOWNSAMP.getKey()) ||
-            e.getKey().equals(BILIN_UPSAMP.getKey()))
-        {
-            BILIN_UPPER = (BILIN_UPSAMP.get() ? 2*MAX_ZOOM.get() : (BILIN_DOWNSAMP.get() ? 0.5 : 0));
-            BILIN_LOWER = (BILIN_DOWNSAMP.get() ? 0 : 1);
-        }
-    }
-
-    /** Manage the visible rectangle of an image with full bounds stored in init. **/
+            e.getKey().equals(BILIN_UPSAMP.getKey())) {
+            bilinUpper = (BILIN_UPSAMP.get() ? 2*MAX_ZOOM.get() : (BILIN_DOWNSAMP.get() ? 0.5 : 0));
+            bilinLower = (BILIN_DOWNSAMP.get() ? 0 : 1);
+        }
+    }
+
+    /**
+     * Manage the visible rectangle of an image with full bounds stored in init.
+     * @since 13127
+     */
     public static class VisRect extends Rectangle {
         private final Rectangle init;
 
+        /**
+         * Constructs a new {@code VisRect}.
+         * @param     x the specified X coordinate
+         * @param     y the specified Y coordinate
+         * @param     width  the width of the rectangle
+         * @param     height the height of the rectangle
+         */
         public VisRect(int x, int y, int width, int height) {
             super(x, y, width, height);
@@ -122 +130 @@
         }
 
+        /**
+         * Constructs a new {@code VisRect} from another one.
+         * @param v rectangle to copy
+         */
         public VisRect(VisRect v) {
             super(v);
@@ -127 +139 @@
         }
 
+        /**
+         * Constructs a new empty {@code VisRect}.
+         */
         public VisRect() {
             this(0, 0, 0, 0);
@@ -269 +284 @@
 
         private boolean mouseIsDragging(MouseEvent e) {
-            return (DRAG_BUTTON == 1 && SwingUtilities.isLeftMouseButton(e)) ||
-                   (DRAG_BUTTON == 2 && SwingUtilities.isMiddleMouseButton(e)) ||
-                   (DRAG_BUTTON == 3 && SwingUtilities.isRightMouseButton(e));
+            return (dragButton == 1 && SwingUtilities.isLeftMouseButton(e)) ||
+                   (dragButton == 2 && SwingUtilities.isMiddleMouseButton(e)) ||
+                   (dragButton == 3 && SwingUtilities.isRightMouseButton(e));
         }
 
         private boolean mouseIsZoomSelecting(MouseEvent e) {
-            return (ZOOM_BUTTON == 1 && SwingUtilities.isLeftMouseButton(e)) ||
-                   (ZOOM_BUTTON == 2 && SwingUtilities.isMiddleMouseButton(e)) ||
-                   (ZOOM_BUTTON == 3 && SwingUtilities.isRightMouseButton(e));
+            return (zoomButton == 1 && SwingUtilities.isLeftMouseButton(e)) ||
+                   (zoomButton == 2 && SwingUtilities.isMiddleMouseButton(e)) ||
+                   (zoomButton == 3 && SwingUtilities.isRightMouseButton(e));
         }
 
@@ -636 +651 @@
             double scale = target.width / (double) r.width; // pixel ratio is 1:1
 
-            if (selectedRect == null && BILIN_LOWER < scale && scale < BILIN_UPPER) {
+            if (selectedRect == null && bilinLower < scale && scale < bilinUpper) {
                 BufferedImage bi = ImageProvider.toBufferedImage(image, r);
                 r.x = r.y = 0;