Remove compiler conditions

Author: JimBobSquarePants

src/ImageSharp.Drawing/Shapes/Path.cs

@@ -419,16 +419,6 @@ private static ReadOnlySpan<char> TrimSeparator(ReadOnlySpan<char> data)
         return data.Slice(idx);
     }
 
-#if !NETCOREAPP2_1_OR_GREATER
-    private static unsafe float ParseFloat(ReadOnlySpan<char> str)
-    {
-        fixed (char* p = str)
-        {
-            return float.Parse(new string(p, 0, str.Length));
-        }
-    }
-#else
     private static float ParseFloat(ReadOnlySpan<char> str)
         => float.Parse(str, provider: CultureInfo.InvariantCulture);
-#endif
 }

src/ImageSharp.Drawing/Shapes/PolygonClipper/Clipper.cs

@@ -44,12 +44,7 @@ public IPath[] GenerateClippedShapes(ClippingOperation operation, IntersectionRu
 
             for (int j = 0; j < path.Count; j++)
             {
-#if NET472
-                Vector2 v = path[j];
-                points[j] = new PointF((float)(v.X / (double)ScalingFactor), (float)(v.Y / (double)ScalingFactor));
-#else
                 points[j] = path[j] / ScalingFactor;
-#endif
             }
 
             shapes[index++] = new Polygon(points);
@@ -62,12 +57,7 @@ public IPath[] GenerateClippedShapes(ClippingOperation operation, IntersectionRu
 
             for (int j = 0; j < path.Count; j++)
             {
-#if NET472
-                Vector2 v = path[j];
-                points[j] = new PointF((float)(v.X / (double)ScalingFactor), (float)(v.Y / (double)ScalingFactor));
-#else
                 points[j] = path[j] / ScalingFactor;
-#endif
             }
 
             shapes[index++] = new Polygon(points);

src/ImageSharp.Drawing/Shapes/PolygonClipper/ClipperOffset.cs

@@ -39,12 +39,7 @@ public ComplexPolygon Execute(float width)
             var points = new PointF[pt.Count];
             for (int j = 0; j < pt.Count; j++)
             {
-#if NET472
-                Vector2 v = pt[j];
-                points[j] = new PointF((float)(v.X / (double)ScalingFactor), (float)(v.Y / (double)ScalingFactor));
-#else
                 points[j] = pt[j] / ScalingFactor;
-#endif
             }
 
             polygons[i] = new Polygon(points);

src/ImageSharp.Drawing/Shapes/PolygonClipper/PolygonOffsetter.cs

@@ -77,7 +77,7 @@ public void Execute(float delta, PathsF solution)
         this.ExecuteInternal(delta);
         if (this.groupList.Count == 0)
         {
-            goto Error;
+            return;
         }
 
         // Clean up self-intersections.
@@ -99,8 +99,6 @@ public void Execute(float delta, PathsF solution)
             clipper.Execute(ClippingOperation.Union, FillRule.Positive, solution);
         }
 
-        Error:
-
         // PolygonClipper will throw for unhandled exceptions but we need to explicitly capture an empty result.
         if (solution.Count == 0)
         {

src/ImageSharp.Drawing/Utilities/NumericUtilities.cs

@@ -38,50 +38,6 @@ public static void AddToAllElements(this Span<float> span, float value)
         }
     }
 
-    // https://apisof.net/catalog/System.Numerics.BitOperations.Log2(UInt32)
-    // BitOperations.Log2() has been introduced in .NET Core 3.0,
-    // since we do target only 3.1+, we can detect it's presence by using SUPPORTS_RUNTIME_INTRINSICS
-    // TODO: Ideally this should have a separate definition in Build.props, but that adaption should be done cross-repo. Using a workaround until then.
-#if SUPPORTS_RUNTIME_INTRINSICS
-    [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public static int Log2(uint value) => BitOperations.Log2(value);
-
-#else
-
-#pragma warning disable SA1515, SA1414, SA1114, SA1201
-    private static ReadOnlySpan<byte> Log2DeBruijn => new byte[32]
-    {
-        00, 09, 01, 10, 13, 21, 02, 29,
-        11, 14, 16, 18, 22, 25, 03, 30,
-        08, 12, 20, 28, 15, 17, 24, 07,
-        19, 27, 23, 06, 26, 05, 04, 31
-    };
-
-    // Adapted from:
-    // https://github.com/dotnet/runtime/blob/5c65d891f203618245184fa54397ced0a8ca806c/src/libraries/System.Private.CoreLib/src/System/Numerics/BitOperations.cs#L205-L223
-    [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public static int Log2(uint value)
-    {
-        // No AggressiveInlining due to large method size
-        // Has conventional contract 0->0 (Log(0) is undefined)
-
-        // Fill trailing zeros with ones, eg 00010010 becomes 00011111
-        value |= value >> 01;
-        value |= value >> 02;
-        value |= value >> 04;
-        value |= value >> 08;
-        value |= value >> 16;
-
-        // uint.MaxValue >> 27 is always in range [0 - 31] so we use Unsafe.AddByteOffset to avoid bounds check
-        return Unsafe.AddByteOffset(
-            // Using deBruijn sequence, k=2, n=5 (2^5=32) : 0b_0000_0111_1100_0100_1010_1100_1101_1101u
-            ref MemoryMarshal.GetReference(Log2DeBruijn),
-            // uint|long -> IntPtr cast on 32-bit platforms does expensive overflow checks not needed here
-            (IntPtr)(int)((value * 0x07C4ACDDu) >> 27));
-    }
-
-#pragma warning restore
-#endif
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static float ClampFloat(float value, float min, float max)
     {

src/ImageSharp.Drawing/Utilities/SortUtility.KeyValueSort.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Six Labors.
 // Licensed under the Six Labors Split License.
 
+using System.Numerics;
 using System.Runtime.CompilerServices;
 
 namespace SixLabors.ImageSharp.Drawing.Utilities;
@@ -44,7 +45,7 @@ private static void IntrospectiveSort(Span<float> keys, Span<TValue> values)
         {
             if (keys.Length > 1)
             {
-                IntroSort(keys, values, 2 * (NumericUtilities.Log2((uint)keys.Length) + 1));
+                IntroSort(keys, values, 2 * (BitOperations.Log2((uint)keys.Length) + 1));
             }
         }