More fixes.

Author: genaray

Arch.LowLevel/Arch.LowLevel.csproj

@@ -13,7 +13,7 @@
 
         <PackageId>Arch.LowLevel</PackageId>
         <Title>Arch.LowLevel</Title>
-        <Version>1.1.4</Version>
+        <Version>1.1.5</Version>
         <Authors>genaray</Authors>
         <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>
         <Description>LowLevel tools for arch.</Description>

Arch.LowLevel/Jagged/SparseJaggedArray.cs

@@ -13,20 +13,26 @@ namespace Arch.LowLevel.Jagged;
 /// <typeparam name="T"></typeparam>
 public record struct SparseBucket<T>
 {
-    
     /// <summary>
     ///     The items array.
     /// </summary>
     internal T[] Array = System.Array.Empty<T>();
+    
+    /// <summary>
+    ///     The filler, the default value.
+    /// </summary>
+    private readonly T _filler;
 
     /// <summary>
     ///     Creates an instance of the <see cref="Bucket{T}"/>.
     /// </summary>
     /// <param name="capacity">The total capacity.</param>
+    /// <param name="filler">The filler.</param>
     /// <param name="allocate">If it should allocate straight forward.</param>
-    public SparseBucket(int capacity, bool allocate = false)
+    public SparseBucket(int capacity, T filler, bool allocate = false)
     {
         Capacity = capacity;
+        _filler = filler;
         if (allocate)
         {
             EnsureCapacity();
@@ -79,6 +85,7 @@ internal void EnsureCapacity()
         }
 
         Array = new T[Capacity];
+        Clear();
     }
 
     /// <summary>
@@ -106,12 +113,12 @@ public ref T this[int i]
     }
 
     /// <summary>
-    ///     Clears this <see cref="SparseBucket{T}"/> and sets all values to the <see cref="filler"/>.
+    ///     Clears this <see cref="SparseBucket{T}"/> and sets all values to the <see cref="_filler"/>.
     /// </summary>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public void Clear(T filler = default)
+    public void Clear()
     {
-        System.Array.Fill(Array, filler);
+        System.Array.Fill(Array, _filler);
     }
 }
 
@@ -165,9 +172,9 @@ public SparseJaggedArray(int bucketSize, int capacity = 64)
         // Fill buckets
         for (var i = 0; i < _buckets.Length; i++)
         {
-            var bucket = new SparseBucket<T>(_bucketSize);
+            var bucket = new SparseBucket<T>(_bucketSize, _filler);
             SetBucket(i, in bucket);
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 
@@ -189,9 +196,9 @@ public SparseJaggedArray(int bucketSize, T filler, int capacity = 64) : this(buc
         // Fill buckets
         for (var i = 0; i < _buckets.Length; i++)
         {
-            var bucket = new SparseBucket<T>(_bucketSize);
+            var bucket = new SparseBucket<T>(_bucketSize, filler);
             SetBucket(i, in bucket);
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 
@@ -363,9 +370,9 @@ public void EnsureCapacity(int newCapacity)
 
         for (var i = length; i < _buckets.Length; i++)
         {
-            var bucket = new SparseBucket<T>(_bucketSize);
+            var bucket = new SparseBucket<T>(_bucketSize, _filler);
             SetBucket(i, bucket);
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 
@@ -457,7 +464,7 @@ public void Clear()
                 continue;
             }
 
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 }

Arch.LowLevel/Jagged/UnsafeSparseJaggedArray.cs

@@ -17,15 +17,23 @@ public record struct UnsafeSparseBucket<T> : IDisposable where T : unmanaged
     ///     The items array.
     /// </summary>
     internal UnsafeArray<T> Array = UnsafeArray.Empty<T>();
+    
+    /// <summary>
+    ///     The filler, the default value.
+    /// </summary>
+    private readonly T _filler;
+
 
     /// <summary>
     ///     Creates an instance of the <see cref="Bucket{T}"/>.
     /// </summary>
     /// <param name="capacity">The total capacity.</param>
+    /// <param name="filler">The filler.</param>
     /// <param name="allocate">If it should allocate straight forward.</param>
-    public UnsafeSparseBucket(int capacity, bool allocate = false)
+    public UnsafeSparseBucket(int capacity, T filler, bool allocate = false)
     {
         Capacity = capacity;
+        _filler = filler;
         if (allocate)
         {
             EnsureCapacity();
@@ -78,6 +86,7 @@ internal void EnsureCapacity()
         }
 
         Array = new UnsafeArray<T>(Capacity);
+        Clear();
     }
 
     /// <summary>
@@ -108,9 +117,9 @@ public ref T this[int i]
     ///     Clears this <see cref="UnsafeSparseBucket{T}"/> and sets all values to the <see cref="filler"/>.
     /// </summary>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public void Clear(T filler = default)
+    public void Clear()
     {
-        UnsafeArray.Fill(ref Array, filler);
+        UnsafeArray.Fill(ref Array, _filler);
     }
 
 
@@ -174,9 +183,9 @@ public UnsafeSparseJaggedArray(int bucketSize, int capacity = 64)
         // Fill buckets
         for (var i = 0; i < _bucketArray.Length; i++)
         {
-            var bucket = new UnsafeSparseBucket<T>(_bucketSize);
+            var bucket = new UnsafeSparseBucket<T>(_bucketSize, _filler);
             SetBucket(i, in bucket);
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 
@@ -198,9 +207,9 @@ public UnsafeSparseJaggedArray(int bucketSize, T filler, int capacity = 64) : th
         // Fill buckets
         for (var i = 0; i < _bucketArray.Length; i++)
         {
-            var bucket = new UnsafeSparseBucket<T>(_bucketSize);
+            var bucket = new UnsafeSparseBucket<T>(_bucketSize, _filler);
             SetBucket(i, in bucket);
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 
@@ -372,9 +381,9 @@ public void EnsureCapacity(int newCapacity)
 
         for (var i = length; i < _bucketArray.Length; i++)
         {
-            var bucket = new UnsafeSparseBucket<T>(_bucketSize);
+            var bucket = new UnsafeSparseBucket<T>(_bucketSize, _filler);
             SetBucket(i, in bucket);
-            bucket.Clear(_filler);
+            bucket.Clear();
         }
     }
 

README.md

@@ -16,7 +16,7 @@ Download the packages and get started today!
 dotnet add package Arch.System --version 1.1.0
 dotnet add package Arch.System.SourceGenerator --version 2.0.0
 dotnet add package Arch.EventBus --version 1.0.2
-dotnet add package Arch.LowLevel --version 1.1.4
+dotnet add package Arch.LowLevel --version 1.1.5
 dotnet add package Arch.Relationships --version 1.0.0
 dotnet add package Arch.Persistence --version 2.0.0
 dotnet add package Arch.AOT.SourceGenerator --version 1.0.1