add some test cases.

Author: overcat

color.go

@@ -13,50 +13,46 @@ type RGB struct {
 func (c *HSV) RGB() *RGB {
 	var r, g, b float64
 	if c.S == 0 {
-		r = c.V * 255
-		g = c.V * 255
-		b = c.V * 255
-	} else {
-		h := c.H * 6
-		if h == 6 {
-			h = 0
+		return &RGB{
+			R: c.V,
+			G: c.V,
+			B: c.V,
 		}
-		i := math.Floor(h)
-		v1 := c.V * (1 - c.S)
-		v2 := c.V * (1 - c.S*(h-i))
-		v3 := c.V * (1 - c.S*(1-(h-i)))
+	}
 
-		switch i {
-		case 0:
-			r = c.V
-			g = v3
-			b = v1
-		case 1:
-			r = v2
-			g = c.V
-			b = v1
-		case 2:
-			r = v1
-			g = c.V
-			b = v3
-		case 3:
-			r = v1
-			g = v2
-			b = c.V
-		case 4:
-			r = v3
-			g = v1
-			b = c.V
-		default:
-			r = c.V
-			g = v1
-			b = v2
-		}
+	i := math.Floor(c.H * 6.0)
+	f := c.H*6.0 - i
+	p := c.V * (1 - c.S)
+	q := c.V * (1 - c.S*f)
+	t := c.V * (1 - c.S*(1-f))
 
-		r = r * 255
-		g = g * 255
-		b = b * 255
+	switch int(i) % 6 {
+	case 0:
+		r = c.V
+		g = t
+		b = p
+	case 1:
+		r = q
+		g = c.V
+		b = p
+	case 2:
+		r = p
+		g = c.V
+		b = t
+	case 3:
+		r = p
+		g = q
+		b = c.V
+	case 4:
+		r = t
+		g = p
+		b = c.V
+	case 5:
+		r = c.V
+		g = p
+		b = q
 	}
+
 	rgb := &RGB{r, g, b}
 	return rgb
 }