fix(bulk-model-sync-gradle): avoid copy of file list

Author: mhuster23

bulk-model-sync-gradle/src/main/kotlin/org/modelix/model/sync/bulk/gradle/tasks/ImportIntoModelServer.kt

@@ -74,7 +74,7 @@ abstract class ImportIntoModelServer @Inject constructor(of: ObjectFactory) : De
             client.runWrite(branchRef) { rootNode ->
                 logger.info("Got root node: {}", rootNode)
                 logger.info("Importing...")
-                ModelImporter(rootNode).importFilesAsRootChildren(*files.toTypedArray())
+                ModelImporter(rootNode).importFilesAsRootChildren(files)
                 logger.info("Import finished")
             }
         }

bulk-model-sync-lib/src/commonMain/kotlin/org/modelix/model/sync/bulk/Util.kt

@@ -21,6 +21,11 @@ import org.modelix.model.api.BuiltinLanguages
 import org.modelix.model.data.ModelData
 import org.modelix.model.data.NodeData
 
+fun mergeModelData(models: Collection<ModelData>): ModelData {
+    return ModelData(root = NodeData(children = models.map { it.root }))
+}
+
+@Deprecated("use collection parameter for better performance")
 fun mergeModelData(vararg models: ModelData): ModelData {
     return ModelData(root = NodeData(children = models.map { it.root }))
 }

bulk-model-sync-lib/src/jvmMain/kotlin/org/modelix/model/sync/bulk/PlatformSpecific.kt

@@ -35,6 +35,12 @@ fun ModelImporter.import(jsonFile: File) {
     import(data)
 }
 
+fun ModelImporter.importFilesAsRootChildren(files: Collection<File>) {
+    val models = files.map { ModelData.fromJson(it.readText()) }
+    import(mergeModelData(models))
+}
+
+@Deprecated("use collection parameter for better performance")
 fun ModelImporter.importFilesAsRootChildren(vararg files: File) {
     val models = files.map { ModelData.fromJson(it.readText()) }
     import(mergeModelData(*models.toTypedArray()))