From 0e229e2bc80c9db20a39ec0c21584d01ce3f6753 Mon Sep 17 00:00:00 2001
From: James Jackson-South <james_south@hotmail.com>
Date: Fri, 28 Apr 2017 11:13:51 +1000
Subject: [PATCH] Fix ConvolutionProcessor

---
 .../Convolution/ConvolutionProcessor.cs       | 88 +++++++++----------
 1 file changed, 44 insertions(+), 44 deletions(-)

diff --git a/src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor.cs b/src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor.cs
index a0c140028..475f14ccf 100644
--- a/src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor.cs
+++ b/src/ImageSharp/Processing/Processors/Convolution/ConvolutionProcessor.cs
@@ -46,51 +46,51 @@ namespace ImageSharp.Processing.Processors
             int maxX = endX - 1;
 
             using (PixelAccessor<TPixel> targetPixels = new PixelAccessor<TPixel>(source.Width, source.Height))
+            using (PixelAccessor<TPixel> sourcePixels = source.Lock())
             {
-                using (PixelAccessor<TPixel> sourcePixels = source.Lock())
-                {
-                    Parallel.For(
-                    startY,
-                    endY,
-                    this.ParallelOptions,
-                    y =>
-                    {
-                        for (int x = startX; x < endX; x++)
-                        {
-                            float red = 0;
-                            float green = 0;
-                            float blue = 0;
-
-                            // Apply each matrix multiplier to the color components for each pixel.
-                            for (int fy = 0; fy < kernelLength; fy++)
-                            {
-                                int fyr = fy - radius;
-                                int offsetY = y + fyr;
-
-                                offsetY = offsetY.Clamp(0, maxY);
-
-                                for (int fx = 0; fx < kernelLength; fx++)
-                                {
-                                    int fxr = fx - radius;
-                                    int offsetX = x + fxr;
-
-                                    offsetX = offsetX.Clamp(0, maxX);
-
-                                    Vector4 currentColor = sourcePixels[offsetX, offsetY].ToVector4();
-                                    currentColor *= this.KernelXY[fy, fx];
-
-                                    red += currentColor.X;
-                                    green += currentColor.Y;
-                                    blue += currentColor.Z;
-                                }
-                            }
-
-                            TPixel packed = default(TPixel);
-                            packed.PackFromVector4(new Vector4(red, green, blue, sourcePixels[x, y].ToVector4().W));
-                            targetPixels[x, y] = packed;
-                        }
-                    });
-                }
+                sourcePixels.CopyTo(targetPixels);
+
+                Parallel.For(
+                 startY,
+                 endY,
+                 this.ParallelOptions,
+                 y =>
+                 {
+                     for (int x = startX; x < endX; x++)
+                     {
+                         float red = 0;
+                         float green = 0;
+                         float blue = 0;
+
+                         // Apply each matrix multiplier to the color components for each pixel.
+                         for (int fy = 0; fy < kernelLength; fy++)
+                         {
+                             int fyr = fy - radius;
+                             int offsetY = y + fyr;
+
+                             offsetY = offsetY.Clamp(0, maxY);
+
+                             for (int fx = 0; fx < kernelLength; fx++)
+                             {
+                                 int fxr = fx - radius;
+                                 int offsetX = x + fxr;
+
+                                 offsetX = offsetX.Clamp(0, maxX);
+
+                                 Vector4 currentColor = sourcePixels[offsetX, offsetY].ToVector4();
+                                 currentColor *= this.KernelXY[fy, fx];
+
+                                 red += currentColor.X;
+                                 green += currentColor.Y;
+                                 blue += currentColor.Z;
+                             }
+                         }
+
+                         TPixel packed = default(TPixel);
+                         packed.PackFromVector4(new Vector4(red, green, blue, sourcePixels[x, y].ToVector4().W));
+                         targetPixels[x, y] = packed;
+                     }
+                 });
 
                 source.SwapPixelsBuffers(targetPixels);
             }