A very minor bug fix: this code looks like it is designed to accept

any hue value and do the modulo itself, except it doesn't quite do it in all cases. At least, the "cannot get here" comment was wrong.
evyroga · Oct 6, 2006 · 615842f · 615842f
1 parent 9d57e53
commit 615842f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/Lib/colorsys.py b/Lib/colorsys.py
@@ -117,7 +117,8 @@ def hsv_to_rgb(h, s, v):
     p = v*(1.0 - s)
     q = v*(1.0 - s*f)
     t = v*(1.0 - s*(1.0-f))
-    if i%6 == 0: return v, t, p
+    i = i%6
+    if i == 0: return v, t, p
     if i == 1: return q, v, p
     if i == 2: return p, v, t
     if i == 3: return p, q, v