From e779c8a2aee57a1b705ef327637e1dc98ad9632e Mon Sep 17 00:00:00 2001 From: Michael Nedzelsky Date: Wed, 25 Mar 2015 16:40:37 +0300 Subject: [PATCH] JpsJsModuleUtils: convert to kotlin --- .../kotlin/jps/build/JpsJsModuleUtils.java | 81 +++++++++---------- 1 file changed, 37 insertions(+), 44 deletions(-) diff --git a/jps-plugin/src/org/jetbrains/kotlin/jps/build/JpsJsModuleUtils.java b/jps-plugin/src/org/jetbrains/kotlin/jps/build/JpsJsModuleUtils.java index 8aadbb2ffba..b18835b71d3 100644 --- a/jps-plugin/src/org/jetbrains/kotlin/jps/build/JpsJsModuleUtils.java +++ b/jps-plugin/src/org/jetbrains/kotlin/jps/build/JpsJsModuleUtils.java @@ -14,56 +14,49 @@ * limitations under the License. */ -package org.jetbrains.kotlin.jps.build; +package org.jetbrains.kotlin.jps.build -import com.intellij.util.Consumer; -import org.jetbrains.annotations.NotNull; -import org.jetbrains.jps.incremental.ModuleBuildTarget; -import org.jetbrains.jps.model.java.JavaSourceRootType; -import org.jetbrains.jps.model.java.JpsJavaModuleType; -import org.jetbrains.jps.model.library.JpsLibrary; -import org.jetbrains.jps.model.library.JpsLibraryRoot; -import org.jetbrains.jps.model.library.JpsOrderRootType; -import org.jetbrains.jps.model.module.JpsModule; -import org.jetbrains.jps.model.module.JpsModuleSourceRoot; -import org.jetbrains.jps.util.JpsPathUtil; +import com.intellij.util.Consumer +import org.jetbrains.jps.incremental.ModuleBuildTarget +import org.jetbrains.jps.model.java.JavaSourceRootProperties +import org.jetbrains.jps.model.java.JavaSourceRootType +import org.jetbrains.jps.model.java.JpsJavaModuleType +import org.jetbrains.jps.model.library.JpsOrderRootType +import org.jetbrains.jps.model.module.JpsModule +import org.jetbrains.jps.util.JpsPathUtil +import java.util.ArrayList -import java.util.ArrayList; -import java.util.List; -import java.util.Set; +class JpsJsModuleUtils private() { + companion object { -class JpsJsModuleUtils { - private JpsJsModuleUtils() {} - - @NotNull - static List getLibraryFilesAndDependencies(@NotNull ModuleBuildTarget target) { - List result = new ArrayList(); - getLibraryFiles(target, result); - getDependencyModulesAndSources(target, result); - return result; - } - - static void getLibraryFiles(@NotNull ModuleBuildTarget target, @NotNull List result) { - Set libraries = JpsUtils.getAllDependencies(target).getLibraries(); - for (JpsLibrary library : libraries) { - for (JpsLibraryRoot root : library.getRoots(JpsOrderRootType.COMPILED)) { - result.add(JpsPathUtil.urlToPath(root.getUrl())); - } + fun getLibraryFilesAndDependencies(target: ModuleBuildTarget): List { + val result = ArrayList() + getLibraryFiles(target, result) + getDependencyModulesAndSources(target, result) + return result } - } - static void getDependencyModulesAndSources(@NotNull final ModuleBuildTarget target, @NotNull final List result) { - JpsUtils.getAllDependencies(target).processModules(new Consumer() { - @Override - public void consume(JpsModule module) { - if (module == target.getModule() || module.getModuleType() != JpsJavaModuleType.INSTANCE) return; - - result.add("@" + module.getName()); - - for (JpsModuleSourceRoot root : module.getSourceRoots(JavaSourceRootType.SOURCE)) { - result.add(JpsPathUtil.urlToPath(root.getUrl())); + fun getLibraryFiles(target: ModuleBuildTarget, result: MutableList) { + val libraries = JpsUtils.getAllDependencies(target).getLibraries() + for (library in libraries) { + for (root in library.getRoots(JpsOrderRootType.COMPILED)) { + result.add(JpsPathUtil.urlToPath(root.getUrl())) } } - }); + } + + fun getDependencyModulesAndSources(target: ModuleBuildTarget, result: MutableList) { + JpsUtils.getAllDependencies(target).processModules(object : Consumer { + override fun consume(module: JpsModule) { + if (module == target.getModule() || module.getModuleType() != JpsJavaModuleType.INSTANCE) return + + result.add("@" + module.getName()) + + for (root in module.getSourceRoots(JavaSourceRootType.SOURCE)) { + result.add(JpsPathUtil.urlToPath(root.getUrl())) + } + } + }) + } } }