From f3c471faf0c90dafec88e8fc8e9c8fae7d43f334 Mon Sep 17 00:00:00 2001
From: James Jackson-South <james_south@hotmail.com>
Date: Wed, 24 May 2017 14:21:11 +1000
Subject: [PATCH] Optimize Invert

---
 .../Processors/Effects/ContrastProcessor.cs   |  2 --
 .../Processors/Effects/InvertProcessor.cs     | 35 +++++++++----------
 2 files changed, 16 insertions(+), 21 deletions(-)

diff --git a/src/ImageSharp/Processing/Processors/Effects/ContrastProcessor.cs b/src/ImageSharp/Processing/Processors/Effects/ContrastProcessor.cs
index f5dc68545e..1daead6e50 100644
--- a/src/ImageSharp/Processing/Processors/Effects/ContrastProcessor.cs
+++ b/src/ImageSharp/Processing/Processors/Effects/ContrastProcessor.cs
@@ -71,12 +71,10 @@ namespace ImageSharp.Processing.Processors
                 this.ParallelOptions,
                 y =>
                     {
-                        int offsetY = y - startY;
                         Span<TPixel> row = source.GetRowSpan(y - startY);
 
                         for (int x = minX; x < maxX; x++)
                         {
-                            int offsetX = x - startX;
                             ref TPixel pixel = ref row[x - startX];
 
                             Vector4 vector = pixel.ToVector4().Expand();
diff --git a/src/ImageSharp/Processing/Processors/Effects/InvertProcessor.cs b/src/ImageSharp/Processing/Processors/Effects/InvertProcessor.cs
index a0348970e7..dc0b134421 100644
--- a/src/ImageSharp/Processing/Processors/Effects/InvertProcessor.cs
+++ b/src/ImageSharp/Processing/Processors/Effects/InvertProcessor.cs
@@ -44,27 +44,24 @@ namespace ImageSharp.Processing.Processors
                 startY = 0;
             }
 
-            using (PixelAccessor<TPixel> sourcePixels = source.Lock())
-            {
-                Parallel.For(
-                    minY,
-                    maxY,
-                    this.ParallelOptions,
-                    y =>
+            Parallel.For(
+                minY,
+                maxY,
+                this.ParallelOptions,
+                y =>
+                    {
+                        Span<TPixel> row = source.GetRowSpan(y - startY);
+
+                        for (int x = minX; x < maxX; x++)
                         {
-                            int offsetY = y - startY;
-                            for (int x = minX; x < maxX; x++)
-                            {
-                                int offsetX = x - startX;
-                                Vector4 color = sourcePixels[offsetX, offsetY].ToVector4();
-                                Vector3 vector = inverseVector - new Vector3(color.X, color.Y, color.Z);
+                            ref TPixel pixel = ref row[x - startX];
 
-                                TPixel packed = default(TPixel);
-                                packed.PackFromVector4(new Vector4(vector, color.W));
-                                sourcePixels[offsetX, offsetY] = packed;
-                            }
-                        });
-            }
+                            var vector = pixel.ToVector4();
+                            Vector3 vector3 = inverseVector - new Vector3(vector.X, vector.Y, vector.Z);
+
+                            pixel.PackFromVector4(new Vector4(vector3, vector.W));
+                        }
+                    });
         }
     }
 }
\ No newline at end of file