Color.HSBtoRGB() will wrap hue for us.

It is documented that hue = hue - (float)Math.floor(hue); It's pointless for us to check if it's over 1f.
2018-07-18 12:02:32 -07:00
parent a7a50773de
commit 145660bf0c
1 changed files with 2 additions and 6 deletions
@@ -73,13 +73,9 @@ public class Colorization
     */
    public int recolorColor (float[] hsv)
    {
-        // for hue, we wrap around
+        // hue will be wrapped-around by HSBtoRGB, we don't need to do it
        float hue = hsv[0] + offsets[0];
-        if (hue > 1.0) {
-            hue -= 1.0;
-        }
-
-        // otherwise we clip
+        // the others we must clip
        float sat = Math.min(Math.max(hsv[1] + offsets[1], 0), 1);
        float val = Math.min(Math.max(hsv[2] + offsets[2], 0), 1);