Fix inconsistent use of 'new' keyword in Deconstruct methods

Removed unnecessary use of 'new' in one Deconstruct method and added it to another for consistency with inheritance behavior. This ensures clarity in method overriding and alignment with expected functionality.
louthy · Feb 1, 2025 · e6475ea · e6475ea
1 parent a710b76
commit e6475ea
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/LanguageExt.Core/Immutable Collections/Iterator/Iterator.cs b/LanguageExt.Core/Immutable Collections/Iterator/Iterator.cs
@@ -114,7 +114,7 @@ public Iterable<A> AsIterable() =>
     /// <summary>
     /// Deconstructor
     /// </summary>
-    public new void Deconstruct(out A head, out Iterator<A> tail)
+    public void Deconstruct(out A head, out Iterator<A> tail)
     {
         head = Head;
         tail = Tail;
@@ -560,7 +560,7 @@ public override void Dispose()
     /// </summary>
     public abstract class Cons : Iterator<A>
     {
-        public void Deconstruct(out A head, out Iterator<A> tail)
+        public new void Deconstruct(out A head, out Iterator<A> tail)
         {
             head = Head;
             tail = Tail;