Merge pull request #5 from rameel/fix-arrayview

Fix an issue in `ArrayView<T>` incorrectly returned a mutable reference
rameel · Jul 10, 2024 · 986fbae · 986fbae
2 parents 3cad21e + 39b5a77
commit 986fbae
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Ramstack.Structures/Collections/ArrayView`1.cs b/Ramstack.Structures/Collections/ArrayView`1.cs
@@ -37,7 +37,7 @@ namespace Ramstack.Collections;
     public bool IsDefault => _array is null;
 
     /// <inheritdoc cref="IReadOnlyList{T}.this"/>
-    public ref T this[int index]
+    public ref readonly T this[int index]
     {
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         get
@@ -202,7 +202,7 @@ public ReadOnlyMemory<T> AsMemory() =>
     /// A reference to the element of the <see cref="ArrayView{T}"/> at index zero.
     /// </returns>
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    public ref T GetPinnableReference()
+    public ref readonly T GetPinnableReference()
     {
         // To match the behavior of ReadOnlySpan<T>