Changeset 10179 in josm for trunk/src/org/openstreetmap/josm/gui/layer

Timestamp:

2016-05-11T02:44:10+02:00 (8 years ago)

Author:

Don-vip

Message:

sonar - squid:AssignmentInSubExpressionCheck - Assignments should not be made from within sub-expressions

Location:

trunk/src/org/openstreetmap/josm/gui/layer

Files:

• AbstractTileSourceLayer.java (modified) (1 diff)
• ImageryLayer.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

@@ -1580 +1580 @@
                 int offset = 200;
                 for (String part: cachedTileLoader.getStats().split("\n")) {
-                    myDrawString(g, tr("Cache stats: {0}", part), 50, offset += 15);
-                }
-
+                    offset += 15;
+                    myDrawString(g, tr("Cache stats: {0}", part), 50, offset);
+                }
             }
         }

trunk/src/org/openstreetmap/josm/gui/layer/ImageryLayer.java

@@ -103 +103 @@
 
     public double getPPD() {
-        if (!Main.isDisplayingMapView()) return Main.getProjection().getDefaultZoomInPPD();
+        if (!Main.isDisplayingMapView())
+            return Main.getProjection().getDefaultZoomInPPD();
         ProjectionBounds bounds = Main.map.mapView.getProjectionBounds();
         return Main.map.mapView.getWidth() / (bounds.maxEast - bounds.minEast);
@@ -122 +123 @@
 
     public void displace(double dx, double dy) {
-        setOffset(this.dx += dx, this.dy += dy);
-    }
-
+        this.dx += dx;
+        this.dy += dy;
+        setOffset(this.dx, this.dy);
+    }
+
+    /**
+     * Returns imagery info.
+     * @return imagery info
+     */
     public ImageryInfo getInfo() {
         return info;