From e687f7b10acdd13facceb3e80d2d0cb0fdfb5b07 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Amaury=20Lev=C3=A9?= <amaury.leve@gmail.com>
Date: Thu, 26 Nov 2020 23:12:33 +0100
Subject: [PATCH] Address review comments

---
 src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs | 4 ++--
 .../Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs         | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs b/src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs
index 075ea2d52..efa3e35a4 100644
--- a/src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs
+++ b/src/ImageSharp/Processing/Processors/Transforms/ISwizzler.cs
@@ -17,7 +17,7 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
         /// Applies the swizzle transformation to a given point.
         /// </summary>
         /// <param name="point">Point to transform.</param>
-        /// <param name="newPoint">The transformed point.</param>
-        void Transform(Point point, out Point newPoint);
+        /// <returns>The transformed point.</returns>
+        Point Transform(Point point);
     }
 }
diff --git a/src/ImageSharp/Processing/Processors/Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs b/src/ImageSharp/Processing/Processors/Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs
index c9b9b4570..aab17d292 100644
--- a/src/ImageSharp/Processing/Processors/Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs
+++ b/src/ImageSharp/Processing/Processors/Transforms/SwizzleProcessor{TSwizzler,TPixel}.cs
@@ -29,10 +29,11 @@ namespace SixLabors.ImageSharp.Processing.Processors.Transforms
             Point newPoint;
             for (p.Y = 0; p.Y < source.Height; p.Y++)
             {
+                Span<TPixel> rowSpan = source.GetPixelRowSpan(p.Y);
                 for (p.X = 0; p.X < source.Width; p.X++)
                 {
-                    this.swizzler.Transform(p, out newPoint);
-                    destination[newPoint.X, newPoint.Y] = source[p.X, p.Y];
+                    newPoint = this.swizzler.Transform(p);
+                    destination[newPoint.X, newPoint.Y] = rowSpan[p.X];
                 }
             }
         }