Fixed bugs

Author: gfoidl

src/ImageSharp/Formats/Jpeg/Components/Decoder/ArithmeticScanDecoder.cs

@@ -53,6 +53,8 @@ internal class ArithmeticScanDecoder : IJpegScanDecoder
 
     private ArithmeticDecodingTable[] acDecodingTables;
 
+    private static readonly byte[] FixedBin = { 113, 0, 0, 0 };
+
     private readonly CancellationToken cancellationToken;
 
     private static readonly int[] ArithmeticTable =
@@ -229,13 +231,7 @@ public void InitDecodingTables(List<ArithmeticDecodingTable> arithmeticDecodingT
         }
     }
 
-    private static ref byte GetFixedBinReference()
-    {
-        // This uses C#'s optimization to refer to the static data segment of the assembly.
-        // No allocation occurs.
-        ReadOnlySpan<byte> fixedBin = new byte[] { 113, 0, 0, 0 };
-        return ref MemoryMarshal.GetReference(fixedBin);
-    }
+    private static ref byte GetFixedBinReference() => ref MemoryMarshal.GetArrayDataReference(FixedBin);
 
     /// <summary>
     /// Decodes the entropy coded data.
@@ -765,7 +761,7 @@ private void DecodeBlockProgressiveDc(ArithmeticDecodingComponent component, ref
                     }
                 }
 
-                v += 1;
+                v++;
                 if (sign != 0)
                 {
                     v = -v;
@@ -860,7 +856,7 @@ private void DecodeBlockProgressiveAc(ArithmeticDecodingComponent component, ref
                     }
                 }
 
-                v += 1;
+                v++;
                 if (sign != 0)
                 {
                     v = -v;
@@ -1016,7 +1012,7 @@ private void DecodeBlockBaseline(
                 }
             }
 
-            v += 1;
+            v++;
             if (sign != 0)
             {
                 v = -v;
@@ -1086,7 +1082,7 @@ private void DecodeBlockBaseline(
                 }
             }
 
-            v += 1;
+            v++;
             if (sign != 0)
             {
                 v = -v;

src/ImageSharp/Primitives/Rectangle.cs

@@ -195,7 +195,7 @@ public int Bottom
     /// <param name="rectangle">The rectangle.</param>
     /// <returns>The <see cref="Point"/>.</returns>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public static Point Center(Rectangle rectangle) => new(rectangle.Left + (rectangle.Width & 1), rectangle.Top + (rectangle.Height & 1));     // & 1 is bit-hack for / 2
+    public static Point Center(Rectangle rectangle) => new(rectangle.Left + (rectangle.Width >> 1), rectangle.Top + (rectangle.Height >> 1));     // >> 1 is bit-hack for / 2
 
     /// <summary>
     /// Creates a rectangle that represents the intersection between <paramref name="a"/> and