From 9638bade2e15e250b6ac624bdd6598a91cfe671c Mon Sep 17 00:00:00 2001
From: Martin Evans <martindevans@gmail.com>
Date: Sat, 28 Oct 2023 15:40:01 +0100
Subject: [PATCH] - Added tests covering `Enumerator`. (#164)

- Removed constructor which made it easy to read out of the valid span range.
  - Only used in one place, fixed that call site.
---
 src/Arch/Core/Archetype.cs   |  2 +-
 src/Arch/Core/Enumerators.cs | 16 ----------------
 2 files changed, 1 insertion(+), 17 deletions(-)

diff --git a/src/Arch/Core/Archetype.cs b/src/Arch/Core/Archetype.cs
index 262ded73..716d61f7 100644
--- a/src/Arch/Core/Archetype.cs
+++ b/src/Arch/Core/Archetype.cs
@@ -404,7 +404,7 @@ internal void SetRange<T>(in Slot from, in Slot to, in T? component = default)
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public Enumerator<Chunk> GetEnumerator()
     {
-        return new Enumerator<Chunk>(Chunks.AsSpan(), Size);
+        return new Enumerator<Chunk>(Chunks.AsSpan(0, Size));
     }
 
     /// <summary>
diff --git a/src/Arch/Core/Enumerators.cs b/src/Arch/Core/Enumerators.cs
index 1a6e6416..6c74f9df 100644
--- a/src/Arch/Core/Enumerators.cs
+++ b/src/Arch/Core/Enumerators.cs
@@ -39,22 +39,6 @@ public Enumerator(Span<T> span)
         _index = _length;
     }
 
-    /// <summary>
-    ///     Initializes a new instance of the <see cref="Enumerator{T}"/> struct.
-    /// </summary>
-    /// <param name="span">The <see cref="Span{T}"/> with items to iterate over.</param>
-    /// <param name="length">Its length or size.</param>
-    public Enumerator(Span<T> span, int length)
-    {
-#if NET7_0_OR_GREATER
-        _ptr = ref MemoryMarshal.GetReference(span);
-#else
-        _ptr = new Ref<T>(ref span.DangerousGetReference());
-#endif
-        _length = length;
-        _index = _length;
-    }
-
     /// <summary>
     ///     Moves to the next item.
     /// </summary>