From 4536baeefdb9b7234ca829ac03237c0c5e7683cc Mon Sep 17 00:00:00 2001
From: Ryan Oldenburg <guzba8@gmail.com>
Date: Fri, 4 Dec 2020 12:42:39 -0600
Subject: [PATCH] address todo

---
 src/pixie/paths.nim | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/pixie/paths.nim b/src/pixie/paths.nim
index f7b4871..f9d3b0a 100644
--- a/src/pixie/paths.nim
+++ b/src/pixie/paths.nim
@@ -587,11 +587,11 @@ proc computeBounds(polys: seq[seq[Vec2]]): Rect =
 {.push checks: off, stacktrace: off.}
 
 proc fillPolygons*(
-    size: Vec2,
-    polys: seq[seq[Vec2]],
-    color: ColorRGBA,
-    quality = 4,
-  ): Image =
+  size: Vec2,
+  polys: seq[seq[Vec2]],
+  color: ColorRGBA,
+  quality = 4,
+): Image =
   const ep = 0.0001 * PI
 
   result = newImage(size.x.int, size.y.int)
@@ -648,11 +648,10 @@ proc fillPolygons*(
           inc curHit
         alphas[x] += penEdge
     for x in 0 ..< result.width:
-      var a = clamp(abs(alphas[x]) / float32(quality), 0.0, 1.0)
+      let a = clamp(abs(alphas[x]) / float32(quality), 0.0, 1.0)
       var colorWithAlpha = color
-      colorWithAlpha.a = uint8(clamp(a, 0, 1) * 255.0)
-      result[x, y] = colorWithAlpha
-      # TODO: don't double-clamp and can probably be unsafe?
+      colorWithAlpha.a = uint8(a * 255.0)
+      result.setRgbaUnsafe(x, y, colorWithAlpha)
 
 {.pop.}