]> git.ktnx.net Git - mobile-ledger.git/commitdiff
work around float weirdness causing 0/360 to become negative
authorDamyan Ivanov <dam+mobileledger@ktnx.net>
Sun, 24 Feb 2019 19:15:13 +0000 (21:15 +0200)
committerDamyan Ivanov <dam+mobileledger@ktnx.net>
Sun, 24 Feb 2019 19:15:33 +0000 (21:15 +0200)
app/src/main/java/net/ktnx/mobileledger/utils/Colors.java

index 7b8352efbe6d25329794b8bf69a460c28adac436..e702d4f8eb917145bbd4d2622dcc9630ef34a0f2 100644 (file)
@@ -59,9 +59,10 @@ public class Colors {
         @ColorLong long result;
         int r, g, b;
 
-        if ((hue < 0) || (hue > 1) || (saturation < 0) || (saturation > 1) || (value < 0) ||
-            (value > 1)) throw new IllegalArgumentException(
-                "hue, saturation, value and alpha must all be between 0 and 1");
+        if ((hue < -0.00005) || (hue > 1.0000005) || (saturation < 0) || (saturation > 1) ||
+            (value < 0) || (value > 1)) throw new IllegalArgumentException(String.format(
+                "hue, saturation, value and alpha must all be between 0 and 1. Arguments given: " +
+                "hue=%1.5f, sat=%1.5f, val=%1.5f", hue, saturation, value));
 
         int h = (int) (hue * 6);
         float f = hue * 6 - h;
@@ -99,6 +100,8 @@ public class Colors {
     }
     public static @ColorInt
     int getPrimaryColorForHue(int degrees) {
+        // 0/360f becomes -0.000something for some reason
+        if (degrees == 0) return getPrimaryColorForHue(0f);
         return getPrimaryColorForHue(degrees / 360f);
     }
     public static @ColorInt