Use generic math in FirstIsToTheLeftOfSecond and `FirstIsToTheRight…

…OfSecond`
linksplatform · Jan 7, 2023 · 5b9d659 · 5b9d659
1 parent 987295f
commit 5b9d659
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/csharp/Platform.Collections.Methods.Tests/RecursionlessSizeBalancedTree.cs b/csharp/Platform.Collections.Methods.Tests/RecursionlessSizeBalancedTree.cs
@@ -58,9 +58,8 @@ public void Free(TElement node)
 
         public bool IsEmpty(TElement node) => EqualityComparer<TreeElement>.Default.Equals(GetElement(node), default);
 
-        protected override bool FirstIsToTheLeftOfSecond(TElement first, TElement second) => Comparer.Compare(first, second) < 0;
-
-        protected override bool FirstIsToTheRightOfSecond(TElement first, TElement second) => Comparer.Compare(first, second) > 0;
+        protected override bool FirstIsToTheLeftOfSecond(TElement first, TElement second) => first < second;
+        protected override bool FirstIsToTheRightOfSecond(TElement first, TElement second) => first > second;
 
         protected override ref TElement GetLeftReference(TElement node) => ref GetElement(node).Left;
 

diff --git a/csharp/Platform.Collections.Methods.Tests/SizeBalancedTree.cs b/csharp/Platform.Collections.Methods.Tests/SizeBalancedTree.cs
@@ -58,9 +58,9 @@ public void Free(TElement node)
 
         public bool IsEmpty(TElement node) => EqualityComparer<TreeElement>.Default.Equals(GetElement(node), default);
 
-        protected override bool FirstIsToTheLeftOfSecond(TElement first, TElement second) => Comparer.Compare(first, second) < 0;
+        protected override bool FirstIsToTheLeftOfSecond(TElement first, TElement second) => first < second;
 
-        protected override bool FirstIsToTheRightOfSecond(TElement first, TElement second) => Comparer.Compare(first, second) > 0;
+        protected override bool FirstIsToTheRightOfSecond(TElement first, TElement second) => first > second;
 
         protected override ref TElement GetLeftReference(TElement node) => ref GetElement(node).Left;
 

diff --git a/csharp/Platform.Collections.Methods.Tests/SizedAndThreadedAVLBalancedTree.cs b/csharp/Platform.Collections.Methods.Tests/SizedAndThreadedAVLBalancedTree.cs
@@ -61,9 +61,9 @@ public void Free(TElement node)
 
         public bool IsEmpty(TElement node) => EqualityComparer<TreeElement>.Default.Equals(GetElement(node), default);
 
-        protected override bool FirstIsToTheLeftOfSecond(TElement first, TElement second) => Comparer.Compare(first, second) < 0;
+        protected override bool FirstIsToTheLeftOfSecond(TElement first, TElement second) => first < second;
 
-        protected override bool FirstIsToTheRightOfSecond(TElement first, TElement second) => Comparer.Compare(first, second) > 0;
+        protected override bool FirstIsToTheRightOfSecond(TElement first, TElement second) => first > second;
 
         protected override sbyte GetBalance(TElement node) => GetElement(node).Balance;