Changeset 8384 in josm for trunk/src/org/openstreetmap/josm/gui/layer/Layer.java

Timestamp:

2015-05-17T15:52:24+02:00 (9 years ago)

Author:

Don-vip

Message:

squid:S1244 - Floating point numbers should not be tested for equality

File:

• trunk/src/org/openstreetmap/josm/gui/layer/Layer.java (modified) (4 diffs)

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

@@ -31 +31 @@
 import org.openstreetmap.josm.tools.Destroyable;
 import org.openstreetmap.josm.tools.ImageProvider;
+import org.openstreetmap.josm.tools.Utils;
 
 /**
@@ -269 +270 @@
         boolean oldValue = isVisible();
         this.visible  = visible;
-        if (visible && opacity == 0) {
+        if (visible && Double.doubleToRawLongBits(opacity) == 0) {
             setOpacity(1);
         } else if (oldValue != isVisible()) {
@@ -281 +282 @@
      */
     public boolean isVisible() {
-        return visible && opacity != 0;
+        return visible && Double.doubleToRawLongBits(opacity) != 0;
     }
 
@@ -294 +295 @@
         boolean oldVisible = isVisible();
         this.opacity = opacity;
-        if (oldOpacity != getOpacity()) {
+        if (!Utils.equalsEpsilon(oldOpacity, getOpacity())) {
             fireOpacityChanged(oldOpacity, getOpacity());
         }