Merge pull request #26 from JetBrains/baseline-perf

Get rid of removeAll to boost baseline calculation performance
JetBrains · Sep 4, 2024 · 30847f9 · 30847f9
2 parents ea5f1fa + a878405
commit 30847f9
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/sarif/src/main/java/com/jetbrains/qodana/sarif/baseline/Baseline.kt b/sarif/src/main/java/com/jetbrains/qodana/sarif/baseline/Baseline.kt
@@ -65,7 +65,10 @@ internal fun applyBaseline(report: Run, baseline: Run, options: Options): DiffSt
             .forEach { result ->
                 //compare with all equal indicators
                 val matchedResults = result.equalIndicators.flatMap(reportIndex::getOrEmpty).toSet()
-                val removed = undecidedFromReport.removeAll(matchedResults)
+                var removed = false
+                for (matchedResult in matchedResults) {
+                    removed = removed || undecidedFromReport.remove(matchedResult)
+                }
 
                 if (removed) {
                     //leads to eliminating problems with the same hash