Merge pull request #483 from CommunityToolkit/dev/no-finalizers

Remove unnecessary finalizers

Author: Sergio0694

CommunityToolkit.HighPerformance/Buffers/ArrayPoolBufferWriter{T}.cs

@@ -99,11 +99,6 @@ public ArrayPoolBufferWriter(ArrayPool<T> pool, int initialCapacity)
         this.index = 0;
     }
 
-    /// <summary>
-    /// Finalizes an instance of the <see cref="ArrayPoolBufferWriter{T}"/> class.
-    /// </summary>
-    ~ArrayPoolBufferWriter() => Dispose();
-
     /// <inheritdoc/>
     Memory<T> IMemoryOwner<T>.Memory
     {
@@ -311,8 +306,6 @@ public void Dispose()
             return;
         }
 
-        GC.SuppressFinalize(this);
-
         this.array = null;
 
         this.pool.Return(array);

CommunityToolkit.HighPerformance/Buffers/MemoryOwner{T}.cs

@@ -77,11 +77,6 @@ private MemoryOwner(int start, int length, ArrayPool<T> pool, T[] array)
         this.array = array;
     }
 
-    /// <summary>
-    /// Finalizes an instance of the <see cref="MemoryOwner{T}"/> class.
-    /// </summary>
-    ~MemoryOwner() => Dispose();
-
     /// <summary>
     /// Gets an empty <see cref="MemoryOwner{T}"/> instance.
     /// </summary>
@@ -295,8 +290,6 @@ public void Dispose()
             return;
         }
 
-        GC.SuppressFinalize(this);
-
         this.array = null;
 
         this.pool.Return(array);