diff --git a/core/descriptor.loader.java/src/org/jetbrains/kotlin/platform/JavaToKotlinClassMap.java b/core/descriptor.loader.java/src/org/jetbrains/kotlin/platform/JavaToKotlinClassMap.java index 0d7598047db..bdcd6acde5c 100644 --- a/core/descriptor.loader.java/src/org/jetbrains/kotlin/platform/JavaToKotlinClassMap.java +++ b/core/descriptor.loader.java/src/org/jetbrains/kotlin/platform/JavaToKotlinClassMap.java @@ -21,7 +21,6 @@ import org.jetbrains.annotations.Nullable; import org.jetbrains.kotlin.builtins.KotlinBuiltIns; import org.jetbrains.kotlin.builtins.PrimitiveType; import org.jetbrains.kotlin.descriptors.ClassDescriptor; -import org.jetbrains.kotlin.descriptors.DeclarationDescriptor; import org.jetbrains.kotlin.load.java.components.TypeUsage; import org.jetbrains.kotlin.name.FqName; import org.jetbrains.kotlin.name.FqNameUnsafe; @@ -37,7 +36,6 @@ public class JavaToKotlinClassMap extends JavaToKotlinClassMapBuilder implements private final Map classDescriptorMap = new HashMap(); private final Map classDescriptorMapForCovariantPositions = new HashMap(); private final Map primitiveTypesMap = new LinkedHashMap(); - private final Map> packagesWithMappedClasses = new HashMap>(); private final Set allKotlinClasses = new LinkedHashSet(); private JavaToKotlinClassMap() { @@ -47,6 +45,7 @@ public class JavaToKotlinClassMap extends JavaToKotlinClassMapBuilder implements private void initPrimitives() { KotlinBuiltIns builtIns = KotlinBuiltIns.getInstance(); + for (JvmPrimitiveType jvmPrimitiveType : JvmPrimitiveType.values()) { PrimitiveType primitiveType = jvmPrimitiveType.getPrimitiveType(); String name = jvmPrimitiveType.getName(); @@ -57,10 +56,10 @@ public class JavaToKotlinClassMap extends JavaToKotlinClassMapBuilder implements primitiveTypesMap.put("[" + name, builtIns.getPrimitiveArrayJetType(primitiveType)); primitiveTypesMap.put(wrapperFqName.asString(), builtIns.getNullablePrimitiveJetType(primitiveType)); } - primitiveTypesMap.put("void", KotlinBuiltIns.getInstance().getUnitType()); + primitiveTypesMap.put("void", builtIns.getUnitType()); for (JetType type : primitiveTypesMap.values()) { - allKotlinClasses.add((ClassDescriptor)type.getConstructor().getDeclarationDescriptor()); + allKotlinClasses.add((ClassDescriptor) type.getConstructor().getDeclarationDescriptor()); } } @@ -71,8 +70,7 @@ public class JavaToKotlinClassMap extends JavaToKotlinClassMapBuilder implements @Nullable public ClassDescriptor mapKotlinClass(@NotNull FqName fqName, @NotNull TypeUsage typeUsage) { - if (typeUsage == TypeUsage.MEMBER_SIGNATURE_COVARIANT - || typeUsage == TypeUsage.SUPERTYPE) { + if (typeUsage == TypeUsage.MEMBER_SIGNATURE_COVARIANT || typeUsage == TypeUsage.SUPERTYPE) { ClassDescriptor descriptor = classDescriptorMapForCovariantPositions.get(fqName); if (descriptor != null) { return descriptor; @@ -106,22 +104,11 @@ public class JavaToKotlinClassMap extends JavaToKotlinClassMapBuilder implements private void register(@NotNull FqName javaClassName, @NotNull ClassDescriptor kotlinDescriptor) { classDescriptorMap.put(javaClassName, kotlinDescriptor); - registerClassInPackage(javaClassName.parent(), kotlinDescriptor); + allKotlinClasses.add(kotlinDescriptor); } private void registerCovariant(@NotNull FqName javaClassName, @NotNull ClassDescriptor kotlinDescriptor) { classDescriptorMapForCovariantPositions.put(javaClassName, kotlinDescriptor); - registerClassInPackage(javaClassName.parent(), kotlinDescriptor); - } - - private void registerClassInPackage(@NotNull FqName packageFqName, @NotNull ClassDescriptor kotlinDescriptor) { - Collection classesInPackage = packagesWithMappedClasses.get(packageFqName); - if (classesInPackage == null) { - classesInPackage = new HashSet(); - packagesWithMappedClasses.put(packageFqName, classesInPackage); - } - classesInPackage.add(kotlinDescriptor); - allKotlinClasses.add(kotlinDescriptor); } @@ -149,19 +136,8 @@ public class JavaToKotlinClassMap extends JavaToKotlinClassMapBuilder implements return mapPlatformClass(className.toSafe()); } - @Override - @NotNull - public Collection mapPlatformClassesInside(@NotNull DeclarationDescriptor containingDeclaration) { - FqNameUnsafe fqName = DescriptorUtils.getFqName(containingDeclaration); - if (!fqName.isSafe()) { - return Collections.emptyList(); - } - Collection result = packagesWithMappedClasses.get(fqName.toSafe()); - return result == null ? Collections.emptySet() : Collections.unmodifiableCollection(result); - } - @NotNull public Set allKotlinClasses() { - return allKotlinClasses; + return Collections.unmodifiableSet(allKotlinClasses); } } diff --git a/core/descriptors/src/org/jetbrains/kotlin/platform/PlatformToKotlinClassMap.java b/core/descriptors/src/org/jetbrains/kotlin/platform/PlatformToKotlinClassMap.java index 53aa813b82c..5fa8c612dae 100644 --- a/core/descriptors/src/org/jetbrains/kotlin/platform/PlatformToKotlinClassMap.java +++ b/core/descriptors/src/org/jetbrains/kotlin/platform/PlatformToKotlinClassMap.java @@ -18,7 +18,6 @@ package org.jetbrains.kotlin.platform; import org.jetbrains.annotations.NotNull; import org.jetbrains.kotlin.descriptors.ClassDescriptor; -import org.jetbrains.kotlin.descriptors.DeclarationDescriptor; import java.util.Collection; import java.util.Collections; @@ -30,17 +29,8 @@ public interface PlatformToKotlinClassMap { public Collection mapPlatformClass(@NotNull ClassDescriptor classDescriptor) { return Collections.emptyList(); } - - @NotNull - @Override - public Collection mapPlatformClassesInside(@NotNull DeclarationDescriptor containingDeclaration) { - return Collections.emptyList(); - } }; @NotNull Collection mapPlatformClass(@NotNull ClassDescriptor classDescriptor); - - @NotNull - Collection mapPlatformClassesInside(@NotNull DeclarationDescriptor containingDeclaration); }