refactor: rename SwiftBucket.Remove to TryRemove

Author: mrdav30

src/SwiftCollections/Collections/SwiftBucket.cs

@@ -179,14 +179,16 @@ public void InsertAt(int index, T item)
         /// <param name="item">The object to remove.</param>
         /// <returns><c>true</c> if item was successfully removed; otherwise, <c>false</c>.</returns>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
-        public bool Remove(T item)
+        public bool TryRemove(T item)
         {
             int index = IndexOf(item);
             if (index < 0) return false;
             RemoveAt(index);
             return true;
         }
 
+        bool ICollection<T>.Remove(T item) => TryRemove(item);
+
         /// <summary>
         /// Removes the item at the specified arrayIndex if it has been allocated.
         /// </summary>

tests/SwiftCollections.Tests/Collections/SwiftBucket.Tests.cs

@@ -61,7 +61,7 @@ public void Remove_ShouldRemoveItemByValue()
             bucket.Add("Item2");
 
             // Act
-            bool removed = bucket.Remove("Item1");
+            bool removed = bucket.TryRemove("Item1");
 
             // Assert
             removed.Should().BeTrue();
@@ -116,7 +116,7 @@ public void Remove_NonExistentItem_ShouldReturnFalse()
             bucket.Add(2);
 
             // Act
-            bool removed = bucket.Remove(3);
+            bool removed = bucket.TryRemove(3);
 
             // Assert
             removed.Should().BeFalse();

tests/SwiftCollections.Tests/Query/SwiftBVH.FixedMathSharp.Tests.cs

@@ -305,7 +305,6 @@ public void StressTest_ResizingIntegrity()
             Assert.Equal(numVolumes, results.Count);
         }
 
-
         [Fact]
         public void MultiThreaded_ConcurrentOperations()
         {