Squeeze out a couple of milliseconds.

Seeing sub 30ms in benchmark for the first time ever.

src/ImageSharp/Formats/Jpeg/Components/Encoder/RgbToYCbCrTables.cs

@@ -104,7 +104,7 @@ namespace ImageSharp.Formats.Jpg
         /// <param name="g">The green value.</param>
         /// <param name="b">The blue value.</param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
-        public static void Allocate(ref float* yBlockRaw, ref float* cbBlockRaw, ref float* crBlockRaw, RgbToYCbCrTables* tables, int index, int r, int g, int b)
+        public static void Allocate(ref float* yBlockRaw, ref float* cbBlockRaw, ref float* crBlockRaw, ref RgbToYCbCrTables* tables, int index, int r, int g, int b)
         {
             // float y = (0.299F * r) + (0.587F * g) + (0.114F * b);
             yBlockRaw[index] = (tables->YRTable[r] + tables->YGTable[g] + tables->YBTable[b]) >> ScaleBits;

src/ImageSharp/Formats/Jpeg/JpegDecoderCore.cs

@@ -7,6 +7,7 @@ namespace ImageSharp.Formats
 {
     using System;
     using System.IO;
+    using System.Runtime.CompilerServices;
     using System.Threading.Tasks;
 
     using ImageSharp.Formats.Jpg;
@@ -680,30 +681,34 @@ namespace ImageSharp.Formats
             using (PixelAccessor<TPixel> pixels = image.Lock())
             {
                 Parallel.For(
-                    0,
+                 0,
-                    image.Height,
+                 image.Height,
-                    image.Configuration.ParallelOptions,
+                 image.Configuration.ParallelOptions,
-                    y =>
+                 y =>
-                    {
+                 {
-                        // TODO. How can we use the fixed tables inside the lambda?
+                     // TODO. This Parallel loop doesn't give us the boost it should.
-                        fixed (YCbCrToRgbTables* tables = &yCbCrToRgbTables)
+                     ref byte ycRef = ref this.ycbcrImage.YChannel.Pixels[0];
-                        {
+                     ref byte cbRef = ref this.ycbcrImage.CbChannel.Pixels[0];
-                            // TODO: Simplify + optimize + share duplicate code across converter methods
+                     ref byte crRef = ref this.ycbcrImage.CrChannel.Pixels[0];
-                            int yo = this.ycbcrImage.GetRowYOffset(y);
+                     fixed (YCbCrToRgbTables* tables = &yCbCrToRgbTables)
-                            int co = this.ycbcrImage.GetRowCOffset(y);
+                     {
-
+                         // TODO: Simplify + optimize + share duplicate code across converter methods
-                            for (int x = 0; x < image.Width; x++)
+                         int yo = this.ycbcrImage.GetRowYOffset(y);
-                            {
+                         int co = this.ycbcrImage.GetRowCOffset(y);
-                                byte yy = this.ycbcrImage.YChannel.Pixels[yo + x];
+
-                                byte cb = this.ycbcrImage.CbChannel.Pixels[co + (x / scale)];
+                         for (int x = 0; x < image.Width; x++)
-                                byte cr = this.ycbcrImage.CrChannel.Pixels[co + (x / scale)];
+                         {
-
+                             int cOff = co + (x / scale);
-                                TPixel packed = default(TPixel);
+                             byte yy = Unsafe.Add(ref ycRef, yo + x);
-                                YCbCrToRgbTables.Pack(ref packed, tables, yy, cb, cr);
+                             byte cb = Unsafe.Add(ref cbRef, cOff);
-                                pixels[x, y] = packed;
+                             byte cr = Unsafe.Add(ref crRef, cOff);
-                            }
+
-                        }
+                             TPixel packed = default(TPixel);
-                    });
+                             YCbCrToRgbTables.Pack(ref packed, tables, yy, cb, cr);
+                             pixels[x, y] = packed;
+                         }
+                     }
+                 });
             }
 
             this.AssignResolution(image);

src/ImageSharp/Formats/Jpeg/JpegEncoderCore.cs

@@ -328,7 +328,7 @@ namespace ImageSharp.Formats
 
                     int index = j8 + i;
 
-                    RgbToYCbCrTables.Allocate(ref yBlockRaw, ref cbBlockRaw, ref crBlockRaw, tables, index, r, g, b);
+                    RgbToYCbCrTables.Allocate(ref yBlockRaw, ref cbBlockRaw, ref crBlockRaw, ref tables, index, r, g, b);
 
                     dataIdx += 3;
                 }