Bug fixes.

Author: genaray

Arch.LowLevel/Arch.LowLevel.csproj

@@ -13,14 +13,15 @@
 
         <PackageId>Arch.LowLevel</PackageId>
         <Title>Arch.LowLevel</Title>
-        <Version>1.1.3</Version>
+        <Version>1.1.4</Version>
         <Authors>genaray</Authors>
         <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>
         <Description>LowLevel tools for arch.</Description>
         <PackageReleaseNotes>Refactored JaggedArrays.
 Increased performance of JaggedArrays.
 Added Array, a new class that acts as a Wrapper around normal generic Arrays for unsafe operations. 
-Added tests. </PackageReleaseNotes>
+Added tests. 
+Bug fixes.</PackageReleaseNotes>
         <PackageTags>c#;.net;.net6;.net7;ecs;game;entity;gamedev; game-development; game-engine; entity-component-system; arch;</PackageTags>
 
         <PackageProjectUrl>https://github.com/genaray/Arch.Extended</PackageProjectUrl>

Arch.LowLevel/Jagged/SparseJaggedArray.cs

@@ -259,9 +259,17 @@ public bool TryGetValue(int index, out T value)
         }
 
         IndexToSlot(index, out var bucketIndex, out var itemIndex);
-        ref var item = ref GetBucket(bucketIndex)[itemIndex];
-
+        
+        // Bucket empty? return false
+        ref var bucket = ref GetBucket(bucketIndex);
+        if (bucket.IsEmpty)
+        {
+            value = _filler;
+            return false;
+        }
+        
         // If the item is the default then the nobody set its value.
+        ref var item = ref bucket[itemIndex];
         if (EqualityComparer<T>.Default.Equals(item, _filler))
         {
             value = _filler;
@@ -289,9 +297,17 @@ public ref T TryGetValue(int index, out bool @bool)
         }
 
         IndexToSlot(index, out var bucketIndex, out var itemIndex);
-        ref var item = ref GetBucket(bucketIndex)[itemIndex];
-
+        
+        // Bucket empty? return false
+        ref var bucket = ref GetBucket(bucketIndex);
+        if (bucket.IsEmpty)
+        {
+            @bool = false;
+            return ref Unsafe.NullRef<T>(); 
+        }
+        
         // If the item is the default then the nobody set its value.
+        ref var item = ref bucket[itemIndex];
         if (EqualityComparer<T>.Default.Equals(item, _filler))
         {
             @bool = false;
@@ -316,8 +332,9 @@ public bool ContainsKey(int index)
         }
 
         IndexToSlot(index, out var bucketIndex, out var itemIndex);
-        ref var bucket = ref GetBucket(bucketIndex);
 
+        // If bucket empty return false
+        ref var bucket = ref GetBucket(bucketIndex);
         if (bucket.IsEmpty)
         {
             return false;

Arch.LowLevel/Jagged/UnsafeSparseJaggedArray.cs

@@ -268,9 +268,17 @@ public bool TryGetValue(int index, out T value)
         }
 
         IndexToSlot(index, out var bucketIndex, out var itemIndex);
-        ref var item = ref GetBucket(bucketIndex)[itemIndex];
+        
+        // Bucket empty? return false
+        ref var bucket = ref GetBucket(bucketIndex);
+        if (bucket.IsEmpty)
+        {
+            value = _filler;
+            return false;
+        }
 
         // If the item is the default then the nobody set its value.
+        ref var item = ref bucket[itemIndex];
         if (EqualityComparer<T>.Default.Equals(item, _filler))
         {
             value = _filler;
@@ -298,9 +306,17 @@ public ref T TryGetValue(int index, out bool @bool)
         }
 
         IndexToSlot(index, out var bucketIndex, out var itemIndex);
-        ref var item = ref GetBucket(bucketIndex)[itemIndex];
-
+        
+        // Bucket empty? return false
+        ref var bucket = ref GetBucket(bucketIndex);
+        if (bucket.IsEmpty)
+        {
+            @bool = false;
+            return ref Unsafe.NullRef<T>(); 
+        }
+        
         // If the item is the default then the nobody set its value.
+        ref var item = ref bucket[itemIndex];
         if (EqualityComparer<T>.Default.Equals(item, _filler))
         {
             @bool = false;
@@ -325,8 +341,9 @@ public bool ContainsKey(int index)
         }
 
         IndexToSlot(index, out var bucketIndex, out var itemIndex);
+        
+        // If bucket empty return false
         ref var bucket = ref GetBucket(bucketIndex);
-    
         if (bucket.IsEmpty)
         {
             return false;

Arch.System.SourceGenerator/Query.cs

@@ -304,7 +304,7 @@ partial class {{queryMethod.ClassName}}{
                     );
 
                     private {{staticModifier}} World? _{{queryMethod.MethodName}}_Initialized;
-                    private {{staticModifier}} Query _{{queryMethod.MethodName}}_Query;
+                    private {{staticModifier}} Query? _{{queryMethod.MethodName}}_Query;
 
                     [MethodImpl(MethodImplOptions.AggressiveInlining)]
                     public {{staticModifier}} void {{queryMethod.MethodName}}Query(World world {{data}}){
@@ -380,7 +380,7 @@ partial class {{queryMethod.ClassName}}{
                     );
 
                     private {{staticModifier}} World? _{{queryMethod.MethodName}}_Initialized;
-                    private {{staticModifier}} Query _{{queryMethod.MethodName}}_Query;
+                    private {{staticModifier}} Query? _{{queryMethod.MethodName}}_Query;
 
                     private struct {{queryMethod.MethodName}}QueryJobChunk : IChunkJob 
                     {

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.3
+dotnet add package Arch.LowLevel --version 1.1.4
 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