From eb120ba1a516132e8a5679c4a26c807c269e3e8f Mon Sep 17 00:00:00 2001
From: Dmitry Pentin <riotbr3aker@gmail.com>
Date: Tue, 30 Nov 2021 08:31:23 +0300
Subject: [PATCH] gfoidl const Size struct jit inlining in color converters

---
 .../Encoder/LuminanceForwardConverter{TPixel}.cs       | 10 +++++-----
 .../Components/Encoder/RgbForwardConverter{TPixel}.cs  | 10 +++++-----
 .../Encoder/YCbCrForwardConverter420{TPixel}.cs        | 10 +++++-----
 .../Encoder/YCbCrForwardConverter444{TPixel}.cs        | 10 +++++-----
 4 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/src/ImageSharp/Formats/Jpeg/Components/Encoder/LuminanceForwardConverter{TPixel}.cs b/src/ImageSharp/Formats/Jpeg/Components/Encoder/LuminanceForwardConverter{TPixel}.cs
index b6edfefed7..6c402fcfd6 100644
--- a/src/ImageSharp/Formats/Jpeg/Components/Encoder/LuminanceForwardConverter{TPixel}.cs
+++ b/src/ImageSharp/Formats/Jpeg/Components/Encoder/LuminanceForwardConverter{TPixel}.cs
@@ -21,11 +21,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
         /// </summary>
         private const int PixelsPerSample = 8 * 8;
 
-        /// <summary>
-        /// <see cref="Size"/> of sampling area from given frame pixel buffer.
-        /// </summary>
-        private static readonly Size SampleSize = new Size(8, 8);
-
         /// <summary>
         /// The Y component
         /// </summary>
@@ -62,6 +57,11 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
             this.config = frame.GetConfiguration();
         }
 
+        /// <summary>
+        /// Gets size of sampling area from given frame pixel buffer.
+        /// </summary>
+        private static Size SampleSize => new(8, 8);
+
         /// <summary>
         /// Converts a 8x8 image area inside 'pixels' at position (x,y) placing the result members of the structure (<see cref="Y"/>)
         /// </summary>
diff --git a/src/ImageSharp/Formats/Jpeg/Components/Encoder/RgbForwardConverter{TPixel}.cs b/src/ImageSharp/Formats/Jpeg/Components/Encoder/RgbForwardConverter{TPixel}.cs
index 0be1076e22..789277d7d3 100644
--- a/src/ImageSharp/Formats/Jpeg/Components/Encoder/RgbForwardConverter{TPixel}.cs
+++ b/src/ImageSharp/Formats/Jpeg/Components/Encoder/RgbForwardConverter{TPixel}.cs
@@ -26,11 +26,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
         /// </summary>
         private const int RgbSpanByteSize = PixelsPerSample * 3;
 
-        /// <summary>
-        /// <see cref="Size"/> of sampling area from given frame pixel buffer.
-        /// </summary>
-        private static readonly Size SampleSize = new Size(8, 8);
-
         /// <summary>
         /// The Red component.
         /// </summary>
@@ -81,6 +76,11 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
             this.config = frame.GetConfiguration();
         }
 
+        /// <summary>
+        /// Gets size of sampling area from given frame pixel buffer.
+        /// </summary>
+        private static Size SampleSize => new(8, 8);
+
         /// <summary>
         /// Converts a 8x8 image area inside 'pixels' at position (x, y) to Rgb24.
         /// </summary>
diff --git a/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter420{TPixel}.cs b/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter420{TPixel}.cs
index bfeafcbb3d..3a878f3c63 100644
--- a/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter420{TPixel}.cs
+++ b/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter420{TPixel}.cs
@@ -25,11 +25,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
         /// </summary>
         private const int RgbSpanByteSize = PixelsPerSample * 3;
 
-        /// <summary>
-        /// <see cref="Size"/> of sampling area from given frame pixel buffer
-        /// </summary>
-        private static readonly Size SampleSize = new Size(16, 8);
-
         /// <summary>
         /// The left Y component
         /// </summary>
@@ -102,6 +97,11 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
             }
         }
 
+        /// <summary>
+        /// Gets size of sampling area from given frame pixel buffer.
+        /// </summary>
+        private static Size SampleSize => new(16, 8);
+
         public void Convert(int x, int y, ref RowOctet<TPixel> currentRows, int idx)
         {
             YCbCrForwardConverter<TPixel>.LoadAndStretchEdges(currentRows, this.pixelSpan, new Point(x, y), SampleSize, this.samplingAreaSize);
diff --git a/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter444{TPixel}.cs b/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter444{TPixel}.cs
index 2dbd1a2dc5..5f7725bddb 100644
--- a/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter444{TPixel}.cs
+++ b/src/ImageSharp/Formats/Jpeg/Components/Encoder/YCbCrForwardConverter444{TPixel}.cs
@@ -25,11 +25,6 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
         /// </summary>
         private const int RgbSpanByteSize = PixelsPerSample * 3;
 
-        /// <summary>
-        /// <see cref="Size"/> of sampling area from given frame pixel buffer
-        /// </summary>
-        private static readonly Size SampleSize = new Size(8, 8);
-
         /// <summary>
         /// The Y component
         /// </summary>
@@ -96,6 +91,11 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components.Encoder
             }
         }
 
+        /// <summary>
+        /// Gets size of sampling area from given frame pixel buffer.
+        /// </summary>
+        private static Size SampleSize => new(8, 8);
+
         /// <summary>
         /// Converts a 8x8 image area inside 'pixels' at position (x,y) placing the result members of the structure (<see cref="Y"/>, <see cref="Cb"/>, <see cref="Cr"/>)
         /// </summary>