From 4a9adda10905ccf9d24f34dc03479cd2ce46c4cc Mon Sep 17 00:00:00 2001
From: James Jackson-South <james_south@hotmail.com>
Date: Wed, 24 May 2017 14:04:47 +1000
Subject: [PATCH] Optimize BackgroundColor

---
 .../Processors/Effects/BackgroundColorProcessor.cs   | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/ImageSharp/Processing/Processors/Effects/BackgroundColorProcessor.cs b/src/ImageSharp/Processing/Processors/Effects/BackgroundColorProcessor.cs
index cc95f15fc..153719191 100644
--- a/src/ImageSharp/Processing/Processors/Effects/BackgroundColorProcessor.cs
+++ b/src/ImageSharp/Processing/Processors/Effects/BackgroundColorProcessor.cs
@@ -6,7 +6,6 @@
 namespace ImageSharp.Processing.Processors
 {
     using System;
-    using System.Numerics;
     using System.Threading.Tasks;
 
     using ImageSharp.Memory;
@@ -64,9 +63,8 @@ namespace ImageSharp.Processing.Processors
 
             int width = maxX - minX;
 
-            using (Buffer<TPixel> colors = new Buffer<TPixel>(width))
-            using (Buffer<float> amount = new Buffer<float>(width))
-            using (PixelAccessor<TPixel> sourcePixels = source.Lock())
+            using (var colors = new Buffer<TPixel>(width))
+            using (var amount = new Buffer<float>(width))
             {
                 for (int i = 0; i < width; i++)
                 {
@@ -81,11 +79,9 @@ namespace ImageSharp.Processing.Processors
                     this.ParallelOptions,
                     y =>
                     {
-                        int offsetY = y - startY;
+                        Span<TPixel> destination = source.GetRowSpan(y - startY).Slice(minX - startX, width);
 
-                        Span<TPixel> destination = sourcePixels.GetRowSpan(offsetY).Slice(minX - startX, width);
-
-                        // this switched color & destination in the 2nd and 3rd places because we are applying the target colour under the current one
+                        // This switched color & destination in the 2nd and 3rd places because we are applying the target colour under the current one
                         blender.Blend(destination, colors, destination, amount);
                     });
             }