diff --git a/compiler/tests/org/jetbrains/kotlin/serialization/builtins/BuiltInsSerializerTest.kt b/compiler/tests/org/jetbrains/kotlin/serialization/builtins/BuiltInsSerializerTest.kt index 520d3bd81a5..7a1c58200b6 100644 --- a/compiler/tests/org/jetbrains/kotlin/serialization/builtins/BuiltInsSerializerTest.kt +++ b/compiler/tests/org/jetbrains/kotlin/serialization/builtins/BuiltInsSerializerTest.kt @@ -20,7 +20,6 @@ import org.jetbrains.kotlin.builtins.KotlinBuiltIns import org.jetbrains.kotlin.builtins.createBuiltInPackageFragmentProvider import org.jetbrains.kotlin.codegen.forTestCompile.ForTestCompileRuntime import org.jetbrains.kotlin.jvm.compiler.LoadDescriptorUtil.TEST_PACKAGE_FQNAME -import org.jetbrains.kotlin.serialization.deserialization.FlexibleTypeCapabilitiesDeserializer import org.jetbrains.kotlin.storage.LockBasedStorageManager import org.jetbrains.kotlin.test.JetTestUtils import org.jetbrains.kotlin.test.TestCaseWithTmpdir @@ -40,13 +39,11 @@ public class BuiltInsSerializerTest : TestCaseWithTmpdir() { val module = JetTestUtils.createEmptyModule("") - val packageFragmentProvider = createBuiltInPackageFragmentProvider( - LockBasedStorageManager(), module, setOf(TEST_PACKAGE_FQNAME), - FlexibleTypeCapabilitiesDeserializer.ThrowException - ) { - val file = File(tmpdir, it) - if (file.exists()) FileInputStream(file) else null - } + val packageFragmentProvider = + createBuiltInPackageFragmentProvider(LockBasedStorageManager(), module, setOf(TEST_PACKAGE_FQNAME)) { + val file = File(tmpdir, it) + if (file.exists()) FileInputStream(file) else null + } module.initialize(packageFragmentProvider) module.addDependencyOnModule(module) diff --git a/core/descriptors/src/org/jetbrains/kotlin/builtins/KotlinBuiltIns.java b/core/descriptors/src/org/jetbrains/kotlin/builtins/KotlinBuiltIns.java index 6d53b29d2d1..09578f0b80f 100644 --- a/core/descriptors/src/org/jetbrains/kotlin/builtins/KotlinBuiltIns.java +++ b/core/descriptors/src/org/jetbrains/kotlin/builtins/KotlinBuiltIns.java @@ -31,7 +31,6 @@ import org.jetbrains.kotlin.name.FqNameUnsafe; import org.jetbrains.kotlin.name.Name; import org.jetbrains.kotlin.resolve.DescriptorUtils; import org.jetbrains.kotlin.resolve.scopes.JetScope; -import org.jetbrains.kotlin.serialization.deserialization.FlexibleTypeCapabilitiesDeserializer; import org.jetbrains.kotlin.storage.LockBasedStorageManager; import org.jetbrains.kotlin.types.*; import org.jetbrains.kotlin.types.checker.JetTypeChecker; @@ -117,7 +116,7 @@ public class KotlinBuiltIns { PackageFragmentProvider packageFragmentProvider = BuiltinsPackage.createBuiltInPackageFragmentProvider( storageManager, builtInsModule, setOf(BUILT_INS_PACKAGE_FQ_NAME, BuiltinsPackage.getKOTLIN_REFLECT_FQ_NAME()), - FlexibleTypeCapabilitiesDeserializer.ThrowException.INSTANCE$, new Function1() { + new Function1() { @Override public InputStream invoke(String path) { return KotlinBuiltIns.class.getClassLoader().getResourceAsStream(path); diff --git a/core/descriptors/src/org/jetbrains/kotlin/builtins/builtInsPackageFragmentProvider.kt b/core/descriptors/src/org/jetbrains/kotlin/builtins/builtInsPackageFragmentProvider.kt index ccd539611ef..62d0f0355bf 100644 --- a/core/descriptors/src/org/jetbrains/kotlin/builtins/builtInsPackageFragmentProvider.kt +++ b/core/descriptors/src/org/jetbrains/kotlin/builtins/builtInsPackageFragmentProvider.kt @@ -16,7 +16,6 @@ package org.jetbrains.kotlin.builtins -import org.jetbrains.kotlin.serialization.deserialization.ResourceLoadingClassDataFinder import org.jetbrains.kotlin.descriptors.ModuleDescriptor import org.jetbrains.kotlin.descriptors.PackageFragmentProvider import org.jetbrains.kotlin.descriptors.PackageFragmentProviderImpl @@ -24,6 +23,7 @@ import org.jetbrains.kotlin.name.FqName import org.jetbrains.kotlin.serialization.deserialization.DeserializationComponents import org.jetbrains.kotlin.serialization.deserialization.FlexibleTypeCapabilitiesDeserializer import org.jetbrains.kotlin.serialization.deserialization.LocalClassResolverImpl +import org.jetbrains.kotlin.serialization.deserialization.ResourceLoadingClassDataFinder import org.jetbrains.kotlin.storage.StorageManager import java.io.InputStream @@ -31,7 +31,6 @@ public fun createBuiltInPackageFragmentProvider( storageManager: StorageManager, module: ModuleDescriptor, packageFqNames: Set, - flexibleTypeCapabilitiesDeserializer: FlexibleTypeCapabilitiesDeserializer, loadResource: (String) -> InputStream? ): PackageFragmentProvider { val packageFragments = packageFqNames.map { fqName -> @@ -48,7 +47,7 @@ public fun createBuiltInPackageFragmentProvider( BuiltInsAnnotationAndConstantLoader(module), provider, localClassResolver, - flexibleTypeCapabilitiesDeserializer + FlexibleTypeCapabilitiesDeserializer.ThrowException ) localClassResolver.setDeserializationComponents(components) diff --git a/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/KotlinJavascriptSerializationUtil.kt b/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/KotlinJavascriptSerializationUtil.kt index 89f86510b99..83227f9daeb 100644 --- a/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/KotlinJavascriptSerializationUtil.kt +++ b/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/KotlinJavascriptSerializationUtil.kt @@ -26,7 +26,6 @@ import org.jetbrains.kotlin.serialization.DescriptorSerializer import org.jetbrains.kotlin.serialization.ProtoBuf import org.jetbrains.kotlin.serialization.SerializationUtil import org.jetbrains.kotlin.serialization.StringTable -import org.jetbrains.kotlin.serialization.deserialization.FlexibleTypeCapabilitiesDeserializer import org.jetbrains.kotlin.storage.StorageManager import org.jetbrains.kotlin.utils.KotlinJavascriptMetadata import org.jetbrains.kotlin.utils.KotlinJavascriptMetadataUtils @@ -65,23 +64,21 @@ public object KotlinJavascriptSerializationUtil { val packageFqNames = getPackages(contentMap).map { FqName(it) }.toSet() if (packageFqNames.isEmpty()) return null - return createKotlinJavascriptPackageFragmentProvider( - storageManager, moduleDescriptor, packageFqNames, FlexibleTypeCapabilitiesDeserializer.Dynamic - ) { + return createKotlinJavascriptPackageFragmentProvider(storageManager, moduleDescriptor, packageFqNames) { path -> - if (!contentMap.containsKey(path)) { - when { - isPackageMetadataFile(path) -> - ByteArrayInputStream(PACKAGE_DEFAULT_BYTES) - isStringTableFile(path) -> - ByteArrayInputStream(STRING_TABLE_DEFAULT_BYTES) - isClassesInPackageFile(path) -> - ByteArrayInputStream(CLASSES_IN_PACKAGE_DEFAULT_BYTES) - else -> - null - } + if (!contentMap.containsKey(path)) { + when { + isPackageMetadataFile(path) -> + ByteArrayInputStream(PACKAGE_DEFAULT_BYTES) + isStringTableFile(path) -> + ByteArrayInputStream(STRING_TABLE_DEFAULT_BYTES) + isClassesInPackageFile(path) -> + ByteArrayInputStream(CLASSES_IN_PACKAGE_DEFAULT_BYTES) + else -> + null } - else ByteArrayInputStream(contentMap.get(path)) + } + else ByteArrayInputStream(contentMap.get(path)) } } diff --git a/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/kotlinJavascriptPackageFragmentProvider.kt b/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/kotlinJavascriptPackageFragmentProvider.kt index cf2f9ac3cbb..aecdc917e55 100644 --- a/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/kotlinJavascriptPackageFragmentProvider.kt +++ b/js/js.serializer/src/org/jetbrains/kotlin/serialization/js/kotlinJavascriptPackageFragmentProvider.kt @@ -16,7 +16,6 @@ package org.jetbrains.kotlin.serialization.js -import org.jetbrains.kotlin.serialization.deserialization.ResourceLoadingClassDataFinder import org.jetbrains.kotlin.descriptors.ModuleDescriptor import org.jetbrains.kotlin.descriptors.PackageFragmentProvider import org.jetbrains.kotlin.descriptors.PackageFragmentProviderImpl @@ -24,6 +23,7 @@ import org.jetbrains.kotlin.name.FqName import org.jetbrains.kotlin.serialization.deserialization.DeserializationComponents import org.jetbrains.kotlin.serialization.deserialization.FlexibleTypeCapabilitiesDeserializer import org.jetbrains.kotlin.serialization.deserialization.LocalClassResolverImpl +import org.jetbrains.kotlin.serialization.deserialization.ResourceLoadingClassDataFinder import org.jetbrains.kotlin.storage.StorageManager import java.io.InputStream @@ -31,7 +31,6 @@ public fun createKotlinJavascriptPackageFragmentProvider( storageManager: StorageManager, module: ModuleDescriptor, packageFqNames: Set, - flexibleTypeCapabilitiesDeserializer: FlexibleTypeCapabilitiesDeserializer, loadResource: (String) -> InputStream? ): PackageFragmentProvider { val packageFragments = packageFqNames.map { fqName -> @@ -48,7 +47,7 @@ public fun createKotlinJavascriptPackageFragmentProvider( KotlinJavascriptAnnotationAndConstantLoader(module), provider, localClassResolver, - flexibleTypeCapabilitiesDeserializer + FlexibleTypeCapabilitiesDeserializer.Dynamic ) localClassResolver.setDeserializationComponents(components)