Remove new ImmArray<T>(params ReadOnlySpan<T>) to revert API break.

Author: oconnor0

Xledger.Collections.Test/TestImmArray.cs

@@ -234,6 +234,7 @@ public void TestNew() {
         var x = new ImmArray<int>(new int[] { 1 });
         Assert.Equal(typeof(ImmArray<int>), x.GetType());
 
+#if NET10_0_OR_GREATER
         var y = new ImmArray<int>(1);
         Assert.Equal(typeof(ImmArray<int>), y.GetType());
 
@@ -248,6 +249,7 @@ public void TestNew() {
 
         var c = new ImmArray<ExplicitIntArray>(new ExplicitIntArray(1));
         Assert.Equal(typeof(ImmArray<ExplicitIntArray>), c.GetType());
+#endif
     }
 
     public record ImplicitIntArray(int i) {

Xledger.Collections/ImmArray.cs

@@ -9,10 +9,11 @@ public static ImmArray<T> Of<T>(params T[] arr) {
 #if NET10_0_OR_GREATER
     [System.Runtime.CompilerServices.OverloadResolutionPriority(1)]
     public static ImmArray<T> Of<T>(params ReadOnlySpan<T> span) {
+        return new ImmArray<T>(span);
 #else
     public static ImmArray<T> Of<T>(ReadOnlySpan<T> span) {
+        return new ImmArray<T>(span.ToArray());
 #endif
-        return new ImmArray<T>(span);
     }
 #endif
 
@@ -77,12 +78,14 @@ public ImmArray(IEnumerable<T> data) {
         }
     }
 
+#if NET10_0_OR_GREATER
     /// <summary>
     /// Constructs an ImmArray by copying the span's contents.
     /// </summary>
     public ImmArray(params ReadOnlySpan<T> data) {
         this.data = data.ToArray();
     }
+#endif
 
     public ReadOnlySpan<T> Span => this.data;