⬇️ Downgrade System.Collections.Immutable (#382)

* 🤢 Replace ImmutableList with ImmutableArray

* ⬇️ Downgrade System.Collections.Immutable

* 💩 Missed two references

Author: tomrijnbeek

Bearded.Utilities.Tests/Algorithms/CoffmanGrahamTests.cs

@@ -133,7 +133,7 @@ public void Solve_PutsChildrenInSeparateLayers()
 
         var solution = solver.Solve(graph);
 
-        for (var i = 0; i < solution.Count; i++)
+        for (var i = 0; i < solution.Length; i++)
         {
             solution[i].Should().ContainSingle().Which.Should().Be(i);
         }

Bearded.Utilities/Algorithms/CoffmanGraham.cs

@@ -37,7 +37,7 @@ public static class CoffmanGraham
 {
     public interface ISolver
     {
-        ImmutableList<ImmutableHashSet<T>> Solve<T>(IDirectedAcyclicGraph<T> graph)
+        ImmutableArray<ImmutableHashSet<T>> Solve<T>(IDirectedAcyclicGraph<T> graph)
             where T : IEquatable<T>;
     }
 
@@ -50,7 +50,7 @@ internal ArbitraryGraphSolver(int maxLayerSize)
             internalSolver = new ReducedGraphSolver(maxLayerSize);
         }
 
-        public ImmutableList<ImmutableHashSet<T>> Solve<T>(IDirectedAcyclicGraph<T> graph)
+        public ImmutableArray<ImmutableHashSet<T>> Solve<T>(IDirectedAcyclicGraph<T> graph)
             where T : IEquatable<T>
         {
             var reducedGraph = DirectedAcyclicGraphTransitiveReducer<T>.ReduceGraph(graph);
@@ -67,10 +67,10 @@ internal ReducedGraphSolver(int maxLayerSize)
             this.maxLayerSize = maxLayerSize;
         }
 
-        public ImmutableList<ImmutableHashSet<T>> Solve<T>(IDirectedAcyclicGraph<T> graph)
+        public ImmutableArray<ImmutableHashSet<T>> Solve<T>(IDirectedAcyclicGraph<T> graph)
             where T : IEquatable<T>
         {
-            if (graph.Count == 0) return ImmutableList<ImmutableHashSet<T>>.Empty;
+            if (graph.Count == 0) return ImmutableArray<ImmutableHashSet<T>>.Empty;
 
             var ordering = createTopologicalOrdering(graph);
             return createLayers(graph, ordering, maxLayerSize);
@@ -125,7 +125,7 @@ DecreasingNumberSequence createDecreasingNumberSequenceOfPredecessorIndices(T e)
             }
         }
 
-        private static ImmutableList<ImmutableHashSet<T>> createLayers<T>(
+        private static ImmutableArray<ImmutableHashSet<T>> createLayers<T>(
             // ReSharper disable once SuggestBaseTypeForParameter
             IDirectedAcyclicGraph<T> graph, IList<T> ordering, int maxLayerSize)
             where T : IEquatable<T>
@@ -169,7 +169,7 @@ private static ImmutableList<ImmutableHashSet<T>> createLayers<T>(
                 elementToLayer.Add(ordering[i], candidateLayer);
             }
 
-            return ImmutableList.CreateRange(layersReversed.Select(b => b.ToImmutable()).Reverse());
+            return ImmutableArray.CreateRange(layersReversed.Select(b => b.ToImmutable()).Reverse());
         }
     }
 

Bearded.Utilities/Bearded.Utilities.csproj

@@ -15,7 +15,7 @@
     <PackageReference Include="OpenTK.Mathematics" Version="4.8.2" />
     <PackageReference Include="OpenTK.Windowing.Desktop" Version="4.8.2" />
     <PackageReference Include="OpenTK.Windowing.GraphicsLibraryFramework" Version="4.8.2" />
-    <PackageReference Include="System.Collections.Immutable" Version="8.0.0" />
+    <PackageReference Include="System.Collections.Immutable" Version="6.0.0" />
     <PackageReference Include="System.Runtime.CompilerServices.Unsafe" Version="6.0.0" />
     <PackageReference Include="System.ValueTuple" Version="4.5.0" />
   </ItemGroup>

Bearded.Utilities/Graphs/AdjacencyListDirectedAcyclicGraph.cs

@@ -7,9 +7,9 @@ sealed class AdjacencyListDirectedAcyclicGraph<T> : AdjacencyListDirectedGraph<T
     where T : IEquatable<T>
 {
     internal AdjacencyListDirectedAcyclicGraph(
-        ImmutableList<T> elements,
-        ImmutableDictionary<T, ImmutableList<T>> directSuccessors,
-        ImmutableDictionary<T, ImmutableList<T>> directPredecessors)
+        ImmutableArray<T> elements,
+        ImmutableDictionary<T, ImmutableArray<T>> directSuccessors,
+        ImmutableDictionary<T, ImmutableArray<T>> directPredecessors)
         : base(
             elements,
             directSuccessors,

Bearded.Utilities/Graphs/AdjacencyListDirectedGraph.cs

@@ -6,17 +6,17 @@ namespace Bearded.Utilities.Graphs;
 
 class AdjacencyListDirectedGraph<T> : IDirectedGraph<T> where T : IEquatable<T>
 {
-    private readonly ImmutableList<T> elements;
-    private readonly ImmutableDictionary<T, ImmutableList<T>> directSuccessors;
-    private readonly ImmutableDictionary<T, ImmutableList<T>> directPredecessors;
+    private readonly ImmutableArray<T> elements;
+    private readonly ImmutableDictionary<T, ImmutableArray<T>> directSuccessors;
+    private readonly ImmutableDictionary<T, ImmutableArray<T>> directPredecessors;
 
     public IEnumerable<T> Elements => elements;
-    public int Count => elements.Count;
+    public int Count => elements.Length;
 
     internal AdjacencyListDirectedGraph(
-        ImmutableList<T> elements,
-        ImmutableDictionary<T, ImmutableList<T>> directSuccessors,
-        ImmutableDictionary<T, ImmutableList<T>> directPredecessors)
+        ImmutableArray<T> elements,
+        ImmutableDictionary<T, ImmutableArray<T>> directSuccessors,
+        ImmutableDictionary<T, ImmutableArray<T>> directPredecessors)
     {
         this.elements = elements;
         this.directSuccessors = directSuccessors;

Bearded.Utilities/Graphs/DirectedGraphBuilder.cs

@@ -7,10 +7,10 @@ namespace Bearded.Utilities.Graphs;
 
 public sealed class DirectedGraphBuilder<T> where T : IEquatable<T>
 {
-    private readonly HashSet<T> elements = new HashSet<T>();
-    private readonly HashSet<T> sources = new HashSet<T>();
-    private readonly Dictionary<T, HashSet<T>> directSuccessors = new Dictionary<T, HashSet<T>>();
-    private readonly Dictionary<T, HashSet<T>> directPredecessors = new Dictionary<T, HashSet<T>>();
+    private readonly HashSet<T> elements = new();
+    private readonly HashSet<T> sources = new();
+    private readonly Dictionary<T, HashSet<T>> directSuccessors = new();
+    private readonly Dictionary<T, HashSet<T>> directPredecessors = new();
 
     public static DirectedGraphBuilder<T> NewBuilder()
     {
@@ -81,13 +81,13 @@ public DirectedGraphBuilder<T> AddArrow(T from, T to)
     public IDirectedGraph<T> CreateGraph()
     {
         return new AdjacencyListDirectedGraph<T>(
-            ImmutableList.CreateRange(elements),
+            ImmutableArray.CreateRange(elements),
             directSuccessors.ToImmutableDictionary(
                 pair => pair.Key,
-                pair => ImmutableList.CreateRange(pair.Value)),
+                pair => ImmutableArray.CreateRange(pair.Value)),
             directPredecessors.ToImmutableDictionary(
                 pair => pair.Key,
-                pair => ImmutableList.CreateRange(pair.Value)));
+                pair => ImmutableArray.CreateRange(pair.Value)));
     }
 
     /// <summary>
@@ -139,12 +139,12 @@ bool leadsBackToCurrentPath(T element)
     public IDirectedAcyclicGraph<T> CreateAcyclicGraphUnsafe()
     {
         return new AdjacencyListDirectedAcyclicGraph<T>(
-            ImmutableList.CreateRange(elements),
+            ImmutableArray.CreateRange(elements),
             directSuccessors.ToImmutableDictionary(
                 pair => pair.Key,
-                pair => ImmutableList.CreateRange(pair.Value)),
+                pair => ImmutableArray.CreateRange(pair.Value)),
             directPredecessors.ToImmutableDictionary(
                 pair => pair.Key,
-                pair => ImmutableList.CreateRange(pair.Value)));
+                pair => ImmutableArray.CreateRange(pair.Value)));
     }
 }