Removed unnecessary code from InsertionOrderSet to further improve th…

…e performance as pointed out by @kpbochenek .
scalatest · Jan 26, 2024 · 7862997 · 7862997
1 parent 128a422
commit 7862997
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/project/GenColCompatHelper.scala b/project/GenColCompatHelper.scala
@@ -75,9 +75,9 @@ object GenColCompatHelper {
           |    def excl(elem: A): scala.collection.immutable.Set[A] = new InsertionOrderSet(elements.filter(_ != elem))
           |    def incl(elem: A): scala.collection.immutable.Set[A] = 
           |      if (underlying.contains(elem)) 
-          |        new InsertionOrderSet(elements) 
+          |        this 
           |      else 
-          |        new InsertionOrderSet(if (underlying.contains(elem)) elements else elements :+ elem)
+          |        new InsertionOrderSet(elements :+ elem)
           |  }
           |}
           |
@@ -149,9 +149,9 @@ object GenColCompatHelper {
           |    def -(elem: A): scala.collection.immutable.Set[A] = new InsertionOrderSet(elements.filter(_ != elem))
           |    def +(elem: A): scala.collection.immutable.Set[A] = 
           |      if (underlying.contains(elem)) 
-          |        new InsertionOrderSet(elements) 
+          |        this
           |      else 
-          |        new InsertionOrderSet(if (underlying.contains(elem)) elements else elements :+ elem)
+          |        new InsertionOrderSet(elements :+ elem)
           |  }
           |}
           |