diff --git a/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.java b/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.java index 8b70d842535..2355b125db1 100644 --- a/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.java +++ b/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.java @@ -344,7 +344,7 @@ public class KotlinToJVMBytecodeCompiler { } else { IncrementalCache incrementalCache = incrementalCacheProvider.getIncrementalCache(moduleId); - packagesWithObsoleteParts = IncrementalPackage.getPackagesWithObsoleteParts(incrementalCache, environment.getSourceFiles()); + packagesWithObsoleteParts = IncrementalPackage.getPackagesWithObsoleteParts(incrementalCache); } BindingTraceContext diagnosticHolder = new BindingTraceContext(); GenerationState generationState = new GenerationState( diff --git a/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalCacheUtil.kt b/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalCacheUtil.kt index 54756b0c404..f82b0df506e 100644 --- a/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalCacheUtil.kt +++ b/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalCacheUtil.kt @@ -17,20 +17,11 @@ package org.jetbrains.kotlin.load.kotlin.incremental import org.jetbrains.kotlin.load.kotlin.incremental.cache.IncrementalCache -import org.jetbrains.kotlin.psi.JetFile import org.jetbrains.kotlin.name.FqName import org.jetbrains.kotlin.resolve.jvm.JvmClassName -import java.util.HashMap -import java.io.File -import org.jetbrains.kotlin.load.kotlin.PackagePartClassUtils -// TODO JetFiles are redundant -public fun IncrementalCache.getPackagesWithObsoleteParts(sourceFilesToCompile: Collection): Collection { - return getObsoletePackageParts(sourceFilesToCompile).map { it.getPackageFqName() } -} - -public fun IncrementalCache.getObsoletePackageParts(sourceFilesToCompile: Collection): Collection { - return getObsoletePackageParts(sourceFilesToCompile.map { File(it.getVirtualFile()!!.getPath()) }).map { JvmClassName.byInternalName(it) } +public fun IncrementalCache.getPackagesWithObsoleteParts(): Collection { + return getObsoletePackageParts().map { JvmClassName.byInternalName(it).getPackageFqName() }.toSet() } public fun IncrementalCache.getPackageData(fqName: FqName): ByteArray? { diff --git a/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalPackageFragmentProvider.kt b/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalPackageFragmentProvider.kt index cbf437fe8c1..48f749997ea 100644 --- a/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalPackageFragmentProvider.kt +++ b/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/IncrementalPackageFragmentProvider.kt @@ -47,7 +47,7 @@ public class IncrementalPackageFragmentProvider( val moduleId: String ) : PackageFragmentProvider { - val obsoletePackageParts = incrementalCache.getObsoletePackageParts(sourceFiles).map { it.getInternalName() }.toSet() + val obsoletePackageParts = incrementalCache.getObsoletePackageParts().toSet() val fqNameToSubFqNames = MultiMap() val fqNameToPackageFragment = HashMap() val fqNamesToLoad: Set @@ -67,7 +67,7 @@ public class IncrementalPackageFragmentProvider( fqNameToPackageFragment[fqName] = IncrementalPackageFragment(fqName) } - fqNamesToLoad = incrementalCache.getPackagesWithObsoleteParts(sourceFiles).toSet() + fqNamesToLoad = obsoletePackageParts.map { JvmClassName.byInternalName(it).getPackageFqName() }.toSet() fqNamesToLoad.forEach { createPackageFragment(it) } } diff --git a/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/cache/IncrementalCache.kt b/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/cache/IncrementalCache.kt index bb66c5b19d9..7a0e14993ee 100644 --- a/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/cache/IncrementalCache.kt +++ b/compiler/frontend.java/src/org/jetbrains/kotlin/load/kotlin/incremental/cache/IncrementalCache.kt @@ -19,7 +19,7 @@ package org.jetbrains.kotlin.load.kotlin.incremental.cache import java.io.File public trait IncrementalCache { - public fun getObsoletePackageParts(sourceFilesToCompile: Collection): Collection + public fun getObsoletePackageParts(): Collection public fun getPackageData(fqName: String): ByteArray? diff --git a/jps-plugin/src/org/jetbrains/kotlin/jps/incremental/IncrementalCacheImpl.kt b/jps-plugin/src/org/jetbrains/kotlin/jps/incremental/IncrementalCacheImpl.kt index 483a57b52a6..047e36da0a1 100644 --- a/jps-plugin/src/org/jetbrains/kotlin/jps/incremental/IncrementalCacheImpl.kt +++ b/jps-plugin/src/org/jetbrains/kotlin/jps/incremental/IncrementalCacheImpl.kt @@ -187,7 +187,7 @@ public class IncrementalCacheImpl(targetDataRoot: File) : StorageOwner, Incremen dirtyOutputClassesMap.clear() } - public override fun getObsoletePackageParts(sourceFilesToCompile: Collection): Collection { + public override fun getObsoletePackageParts(): Collection { return dirtyOutputClassesMap.getDirtyOutputClasses().filter { packagePartMap.isPackagePart(JvmClassName.byInternalName(it)) } }