Fix minor bugs in SortedArrayList and DefaultGroupPlacementStrategy.

homothetyhk · Jan 16, 2022 · 2530467 · 2530467
1 parent ca06a0e
commit 2530467
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/RandomizerCore/Collections/SortedArrayList.cs b/RandomizerCore/Collections/SortedArrayList.cs
@@ -188,6 +188,7 @@ public bool Contains(T item)
         public void CopyTo(T[] array, int arrayIndex)
         {
             ts.CopyTo(array, arrayIndex);
+            Array.Reverse(array, arrayIndex, ts.Count);
         }
 
         /// <summary>

diff --git a/RandomizerCore/Randomization/DefaultGroupPlacementStrategy.cs b/RandomizerCore/Randomization/DefaultGroupPlacementStrategy.cs
@@ -126,7 +126,7 @@ public virtual List<RandoPlacement> PlaceCoupledGroup(CoupledRandomizationGroup
                     }
                     else dualMeanSphereProgressionPriorities.Add(int.MinValue);
 
-                    dualLocations.Add(new SortedArrayList<IRandoLocation>(sd.Locations));
+                    dualLocations.Add(new SortedArrayList<IRandoLocation>(sd.Locations, ComparerUtil.LocationComparer, ComparerUtil.LocationEqualityComparer));
                 }
 
                 SortedArrayList<IRandoItem> remainingItems = new(dualLocations.SelectMany(l => l.Cast<IRandoItem>()), ComparerUtil.ItemComparer, ComparerUtil.ItemEqualityComparer);