diff --git a/compiler/light-classes/src/org/jetbrains/kotlin/asJava/LightClassUtil.kt b/compiler/light-classes/src/org/jetbrains/kotlin/asJava/LightClassUtil.kt index 42a4e8b4b2e..8150070710b 100644 --- a/compiler/light-classes/src/org/jetbrains/kotlin/asJava/LightClassUtil.kt +++ b/compiler/light-classes/src/org/jetbrains/kotlin/asJava/LightClassUtil.kt @@ -33,7 +33,6 @@ import com.intellij.util.SmartList import org.jetbrains.kotlin.builtins.KotlinBuiltIns import org.jetbrains.kotlin.fileClasses.javaFileFacadeFqName import org.jetbrains.kotlin.load.java.JvmAbi -import org.jetbrains.kotlin.load.kotlin.PackageClassUtils import org.jetbrains.kotlin.name.FqName import org.jetbrains.kotlin.psi.* import org.jetbrains.kotlin.psi.psiUtil.getNonStrictParentOfType @@ -141,7 +140,7 @@ public object LightClassUtil { } public fun getLightFieldForCompanionObject(companionObject: KtClassOrObject): PsiField? { - val outerPsiClass = getWrappingClass(companionObject, true) + val outerPsiClass = getWrappingClass(companionObject) if (outerPsiClass != null) { for (fieldOfParent in outerPsiClass.fields) { if ((fieldOfParent is KtLightElement<*, *>) && fieldOfParent.getOrigin() === companionObject) { @@ -163,7 +162,7 @@ public object LightClassUtil { } private fun getLightClassBackingField(declaration: KtDeclaration): PsiField? { - var psiClass: PsiClass = getWrappingClass(declaration, true) ?: return null + var psiClass: PsiClass = getWrappingClass(declaration) ?: return null if (psiClass is KtLightClass) { val origin = psiClass.getOrigin() @@ -203,10 +202,10 @@ public object LightClassUtil { } private fun getPsiMethodWrappers(declaration: KtDeclaration, collectAll: Boolean): List { - val psiClasses = getWrappingClasses(declaration, collectAll) + val psiClass = getWrappingClass(declaration) ?: return emptyList() val methods = SmartList() - for (method in psiClasses.flatMap { it.methods.asList() }) { + for (method in psiClass.methods.asList()) { try { if (method is KtLightMethod && method.getOrigin() === declaration) { methods.add(method) @@ -227,17 +226,7 @@ public object LightClassUtil { return methods } - private fun getWrappingClasses(declaration: KtDeclaration, collectAll: Boolean): Collection { - val wrappingClass = getWrappingClass(declaration, true) - val oldPackagePartWrappingClass = if (declaration.parent is KtFile && collectAll) - getWrappingClass(declaration, false) - else - null - - return setOf(wrappingClass, oldPackagePartWrappingClass).filterNotNull() - } - - private fun getWrappingClass(declaration: KtDeclaration, useNewPackageParts: Boolean): PsiClass? { + private fun getWrappingClass(declaration: KtDeclaration): PsiClass? { var declaration = declaration if (declaration is KtParameter) { val constructorClass = KtPsiUtil.getClassIfParameterIsProperty(declaration) @@ -267,11 +256,7 @@ public object LightClassUtil { if (parent is KtFile) { // top-level declaration - val fqName = if (useNewPackageParts) - parent.javaFileFacadeFqName - else - PackageClassUtils.getPackageClassFqName(parent.packageFqName) - + val fqName = parent.javaFileFacadeFqName val project = declaration.project return JavaElementFinder.getInstance(project).findClass(fqName.asString(), GlobalSearchScope.allScope(project)) } diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/caches/KotlinShortNamesCache.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/caches/KotlinShortNamesCache.kt index 0c1145a5ec8..9d91c2cde60 100644 --- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/caches/KotlinShortNamesCache.kt +++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/caches/KotlinShortNamesCache.kt @@ -30,7 +30,6 @@ import org.jetbrains.kotlin.asJava.defaultImplsChild import org.jetbrains.kotlin.fileClasses.javaFileFacadeFqName import org.jetbrains.kotlin.idea.stubindex.KotlinClassShortNameIndex import org.jetbrains.kotlin.idea.stubindex.KotlinFileFacadeShortNameIndex -import org.jetbrains.kotlin.idea.stubindex.PackageIndexUtil import org.jetbrains.kotlin.load.java.JvmAbi import org.jetbrains.kotlin.name.FqName import java.util.* @@ -43,8 +42,6 @@ public class KotlinShortNamesCache(private val project: Project) : PsiShortNames val classNames = KotlinClassShortNameIndex.getInstance().getAllKeys(project) // package classes can not be indexed, since they have no explicit declarations - val packageClassShortNames = PackageIndexUtil.getAllPossiblePackageClasses(project).keySet() - classNames.addAll(packageClassShortNames) classNames.addAll(KotlinFileFacadeShortNameIndex.INSTANCE.getAllKeys(project)) return classNames.toTypedArray() @@ -56,9 +53,6 @@ public class KotlinShortNamesCache(private val project: Project) : PsiShortNames override fun getClassesByName(name: String, scope: GlobalSearchScope): Array { val allFqNames = HashSet() - val packageClassNames = PackageIndexUtil.getAllPossiblePackageClasses(project).get(name) - allFqNames.addAll(packageClassNames) - KotlinClassShortNameIndex.getInstance().get(name, project, scope) .mapTo(allFqNames) { it.fqName } diff --git a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/stubindex/PackageIndexUtil.kt b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/stubindex/PackageIndexUtil.kt index c3db98bc602..fdaac38013d 100644 --- a/idea/idea-analysis/src/org/jetbrains/kotlin/idea/stubindex/PackageIndexUtil.kt +++ b/idea/idea-analysis/src/org/jetbrains/kotlin/idea/stubindex/PackageIndexUtil.kt @@ -19,14 +19,9 @@ package org.jetbrains.kotlin.idea.stubindex import com.intellij.openapi.project.Project import com.intellij.psi.search.GlobalSearchScope import com.intellij.psi.stubs.StubIndex -import org.jetbrains.kotlin.psi.KtFile import org.jetbrains.kotlin.name.FqName -import com.intellij.util.containers.MultiMap -import org.jetbrains.kotlin.load.kotlin.PackageClassUtils -import com.intellij.psi.util.CachedValuesManager -import com.intellij.psi.util.CachedValueProvider.Result -import com.intellij.psi.util.PsiModificationTracker import org.jetbrains.kotlin.name.Name +import org.jetbrains.kotlin.psi.KtFile public object PackageIndexUtil { @JvmStatic @@ -73,22 +68,4 @@ public object PackageIndexUtil { } return result } - - @JvmStatic - public fun getAllPossiblePackageClasses(project: Project): MultiMap { - return CachedValuesManager.getManager(project).getCachedValue(project) { - Result(computeAllPossiblePackageClasses(project), PsiModificationTracker.OUT_OF_CODE_BLOCK_MODIFICATION_COUNT) - }!! - } - - private fun computeAllPossiblePackageClasses(project: Project): MultiMap { - val packageFqNames = KotlinExactPackagesIndex.getInstance().getAllKeys(project) - val result = MultiMap() - for (packageFqName in packageFqNames) { - val packageClassFqName = PackageClassUtils.getPackageClassFqName(FqName(packageFqName)) - result.putValue(packageClassFqName.shortName().asString(), packageClassFqName) - } - - return result - } } diff --git a/idea/tests/org/jetbrains/kotlin/idea/filters/KotlinExceptionFilterTest.kt b/idea/tests/org/jetbrains/kotlin/idea/filters/KotlinExceptionFilterTest.kt index 93984d63d6d..454997f09b9 100644 --- a/idea/tests/org/jetbrains/kotlin/idea/filters/KotlinExceptionFilterTest.kt +++ b/idea/tests/org/jetbrains/kotlin/idea/filters/KotlinExceptionFilterTest.kt @@ -32,7 +32,6 @@ import com.intellij.testFramework.PlatformTestCase import com.intellij.testFramework.PsiTestUtil import junit.framework.TestCase import org.jetbrains.kotlin.idea.test.PluginTestCaseBase -import org.jetbrains.kotlin.load.kotlin.PackageClassUtils.getPackageClassFqName import org.jetbrains.kotlin.load.kotlin.PackagePartClassUtils import org.jetbrains.kotlin.name.FqName import org.jetbrains.kotlin.test.MockLibraryUtil