Remove a TODO

jbartok · jbartok · commit 560589040167 · 2025-03-20T10:50:43.000+02:00
diff --git a/lsp/src/main/kotlin/org/gradle/declarative/lsp/extension/AnalysisSchemaExtensions.kt b/lsp/src/main/kotlin/org/gradle/declarative/lsp/extension/AnalysisSchemaExtensions.kt
@@ -111,9 +111,7 @@ else {
         val mergedResult = mutableMapOf<FqName, Map<List<DataType.ParameterizedTypeInstance.TypeArgument>, DataType.ClassDataType>>()
         schemas.forEach { schema ->
             schema.genericInstantiationsByFqName.forEach { instantiation ->
-                mergedResult.merge(instantiation.key, instantiation.value) { oldVal, newVal ->
-                    oldVal + newVal // TODO: is this correct?
-                }
+                mergedResult.merge(instantiation.key, instantiation.value) { oldVal, newVal -> oldVal + newVal }
             }
         }
         mergedResult

Original file line number	Diff line number	Diff line change
`@@ -111,9 +111,7 @@ else {`
`111`	`111`	`val mergedResult = mutableMapOf<FqName, Map<List<DataType.ParameterizedTypeInstance.TypeArgument>, DataType.ClassDataType>>()`
`112`	`112`	`schemas.forEach { schema ->`
`113`	`113`	`schema.genericInstantiationsByFqName.forEach { instantiation ->`
`114`		`- mergedResult.merge(instantiation.key, instantiation.value) { oldVal, newVal ->`
`115`		`- oldVal + newVal // TODO: is this correct?`
`116`		`- }`
	`114`	`+ mergedResult.merge(instantiation.key, instantiation.value) { oldVal, newVal -> oldVal + newVal }`
`117`	`115`	`}`
`118`	`116`	`}`
`119`	`117`	`mergedResult`