refactor: Return a set from mergeCopyrightFindings()

Returning a sorted set is no more necessary, because `ScanSummary` has been changed in a preceeding change to use a set for the copyright findings. Signed-off-by: Frank Viernau <[email protected]>
oss-review-toolkit · May 16, 2023 · 587a043 · 587a043
1 parent baf7adf
commit 587a043
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/scanner/src/main/kotlin/provenance/NestedProvenanceScanResult.kt b/scanner/src/main/kotlin/provenance/NestedProvenanceScanResult.kt
@@ -120,12 +120,12 @@ data class NestedProvenanceScanResult(
         return findings
     }
 
-    private fun Map<KnownProvenance, List<ScanResult>>.mergeCopyrightFindings(): SortedSet<CopyrightFinding> {
+    private fun Map<KnownProvenance, List<ScanResult>>.mergeCopyrightFindings(): Set<CopyrightFinding> {
         val findingsByPath = mapKeys { getPath(it.key) }.mapValues { (_, scanResults) ->
             scanResults.flatMap { it.summary.copyrightFindings }
         }
 
-        val findings = findingsByPath.flatMapTo(sortedSetOf()) { (path, findings) ->
+        val findings = findingsByPath.flatMapTo(mutableSetOf()) { (path, findings) ->
             val prefix = if (path.isEmpty()) path else "$path/"
             findings.map { it.copy(location = it.location.copy(path = "$prefix${it.location.path}")) }
         }