From 1223d918163720d5a38f00159e33aa481ed673bb Mon Sep 17 00:00:00 2001
From: Ryan Oldenburg <guzba8@gmail.com>
Date: Fri, 4 Dec 2020 00:26:14 -0600
Subject: [PATCH] inplace blur

---
 src/pixie/images.nim | 14 ++++++--------
 src/pixie/masks.nim  |  4 ++--
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/pixie/images.nim b/src/pixie/images.nim
index 77b2ffe..d230c30 100644
--- a/src/pixie/images.nim
+++ b/src/pixie/images.nim
@@ -207,15 +207,16 @@ proc resize*(srcImage: Image, width, height: int): Image =
     ))
   )
 
-proc blur*(image: Image, radius: float32): Image =
+proc blur*(image: Image, radius: float32) =
   ## Applies Gaussian blur to the image given a radius.
   let radius = round(radius).int
   if radius == 0:
-    return image.copy()
+    return
 
   # Compute lookup table for 1d Gaussian kernel.
-  var lookup = newSeq[float](radius * 2 + 1)
-  var total = 0.0
+  var
+    lookup = newSeq[float](radius * 2 + 1)
+    total = 0.0
   for xb in -radius .. radius:
     let s = radius.float32 / 2.2 # 2.2 matches Figma.
     let x = xb.float32
@@ -246,7 +247,6 @@ proc blur*(image: Image, radius: float32): Image =
       blurX.setRgbaUnsafe(x, y, c.rgba)
 
   # Blur in the Y direction.
-  var blurY = newImage(image.width, image.height)
   for y in 0 ..< image.height:
     for x in 0 ..< image.width:
       var c: Color
@@ -263,9 +263,7 @@ proc blur*(image: Image, radius: float32): Image =
       c.r = c.r / totalA
       c.g = c.g / totalA
       c.b = c.b / totalA
-      blurY.setRgbaUnsafe(x, y, c.rgba)
-
-  return blurY
+      image.setRgbaUnsafe(x, y, c.rgba)
 
 proc blurAlpha*(image: Image, radius: float32) =
   ## Applies Gaussian blur to the image given a radius.
diff --git a/src/pixie/masks.nim b/src/pixie/masks.nim
index 813bae2..16e6b25 100644
--- a/src/pixie/masks.nim
+++ b/src/pixie/masks.nim
@@ -1,3 +1,4 @@
+import system/memory
 
 type
   Mask* = ref object
@@ -51,8 +52,7 @@ proc `[]=`*(mask: Mask, x, y: int, value: uint8) {.inline.} =
 
 proc fill*(mask: Mask, value: uint8) =
   ## Fills the mask with the parameter value.
-  for i in 0 ..< mask.data.len:
-    mask.data[i] = value
+  nimSetMem(mask.data[0].addr, value.cint, mask.data.len)
 
 proc invert*(mask: Mask) =
   ## Inverts the entire mask value.