refactor(utils): Slightly re-write a function

Avoid a conversion to a `SortedSet` and the `getValue()` made in each iteration. Signed-off-by: Frank Viernau <[email protected]>
oss-review-toolkit · May 27, 2024 · d22d659 · d22d659
1 parent 2bfa41f
commit d22d659
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/model/src/main/kotlin/utils/DependencyGraphBuilder.kt b/model/src/main/kotlin/utils/DependencyGraphBuilder.kt
@@ -553,8 +553,8 @@ private fun constructSortedScopeMappings(
 ): Map<String, List<RootDependencyIndex>> {
     val orderedMappings = mutableMapOf<String, List<RootDependencyIndex>>()
 
-    scopeMappings.keys.toSortedSet().forEach { scope ->
-        orderedMappings[scope] = scopeMappings.getValue(scope)
+    scopeMappings.entries.sortedBy { it.key }.forEach { (scope, rootDependencyIndices) ->
+        orderedMappings[scope] = rootDependencyIndices
             .map { it.mapIndex(indexMapping) }
             .sortedWith(rootDependencyIndexComparator)
     }