Rename JvmBuiltInsSettings -> JvmBuiltInsCustomizer
This commit is contained in:
@@ -28,7 +28,6 @@ import org.jetbrains.kotlin.descriptors.ModuleDescriptor
|
|||||||
import org.jetbrains.kotlin.frontend.di.configureIncrementalCompilation
|
import org.jetbrains.kotlin.frontend.di.configureIncrementalCompilation
|
||||||
import org.jetbrains.kotlin.frontend.di.configureModule
|
import org.jetbrains.kotlin.frontend.di.configureModule
|
||||||
import org.jetbrains.kotlin.frontend.di.configureStandardResolveComponents
|
import org.jetbrains.kotlin.frontend.di.configureStandardResolveComponents
|
||||||
import org.jetbrains.kotlin.idea.MainFunctionDetector
|
|
||||||
import org.jetbrains.kotlin.incremental.components.ExpectActualTracker
|
import org.jetbrains.kotlin.incremental.components.ExpectActualTracker
|
||||||
import org.jetbrains.kotlin.incremental.components.LookupTracker
|
import org.jetbrains.kotlin.incremental.components.LookupTracker
|
||||||
import org.jetbrains.kotlin.load.java.AbstractJavaClassFinder
|
import org.jetbrains.kotlin.load.java.AbstractJavaClassFinder
|
||||||
@@ -127,7 +126,7 @@ fun StorageComponentContainer.configureJavaSpecificComponents(
|
|||||||
useInstance(languageVersionSettings.getFlag(JvmAnalysisFlags.javaTypeEnhancementState))
|
useInstance(languageVersionSettings.getFlag(JvmAnalysisFlags.javaTypeEnhancementState))
|
||||||
|
|
||||||
if (useBuiltInsProvider) {
|
if (useBuiltInsProvider) {
|
||||||
useInstance((moduleContext.module.builtIns as JvmBuiltIns).settings)
|
useInstance((moduleContext.module.builtIns as JvmBuiltIns).customizer)
|
||||||
useImpl<JvmBuiltInsPackageFragmentProvider>()
|
useImpl<JvmBuiltInsPackageFragmentProvider>()
|
||||||
}
|
}
|
||||||
useImpl<OptionalAnnotationPackageFragmentProvider>()
|
useImpl<OptionalAnnotationPackageFragmentProvider>()
|
||||||
|
|||||||
@@ -63,8 +63,8 @@ class JvmBuiltIns(storageManager: StorageManager, kind: Kind) : KotlinBuiltIns(s
|
|||||||
this.isAdditionalBuiltInsFeatureSupported = isAdditionalBuiltInsFeatureSupported
|
this.isAdditionalBuiltInsFeatureSupported = isAdditionalBuiltInsFeatureSupported
|
||||||
}
|
}
|
||||||
|
|
||||||
val settings: JvmBuiltInsSettings by storageManager.createLazyValue {
|
val customizer: JvmBuiltInsCustomizer by storageManager.createLazyValue {
|
||||||
JvmBuiltInsSettings(
|
JvmBuiltInsCustomizer(
|
||||||
builtInsModule, storageManager,
|
builtInsModule, storageManager,
|
||||||
{ ownerModuleDescriptor.sure { "JvmBuiltins has not been initialized properly" } },
|
{ ownerModuleDescriptor.sure { "JvmBuiltins has not been initialized properly" } },
|
||||||
{
|
{
|
||||||
@@ -83,9 +83,9 @@ class JvmBuiltIns(storageManager: StorageManager, kind: Kind) : KotlinBuiltIns(s
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun getPlatformDependentDeclarationFilter(): PlatformDependentDeclarationFilter = settings
|
override fun getPlatformDependentDeclarationFilter(): PlatformDependentDeclarationFilter = customizer
|
||||||
|
|
||||||
override fun getAdditionalClassPartsProvider(): AdditionalClassPartsProvider = settings
|
override fun getAdditionalClassPartsProvider(): AdditionalClassPartsProvider = customizer
|
||||||
|
|
||||||
override fun getClassDescriptorFactories() =
|
override fun getClassDescriptorFactories() =
|
||||||
super.getClassDescriptorFactories() + JvmBuiltInClassDescriptorFactory(storageManager, builtInsModule)
|
super.getClassDescriptorFactories() + JvmBuiltInClassDescriptorFactory(storageManager, builtInsModule)
|
||||||
|
|||||||
+3
-1
@@ -45,7 +45,9 @@ import org.jetbrains.kotlin.utils.DFS
|
|||||||
import org.jetbrains.kotlin.utils.SmartSet
|
import org.jetbrains.kotlin.utils.SmartSet
|
||||||
import java.util.*
|
import java.util.*
|
||||||
|
|
||||||
open class JvmBuiltInsSettings(
|
// This class is worth splitting into two implementations of AdditionalClassPartsProvider and PlatformDependentDeclarationFilter
|
||||||
|
// But currently, they shares a piece of code and probably it's better to postpone it
|
||||||
|
class JvmBuiltInsCustomizer(
|
||||||
private val moduleDescriptor: ModuleDescriptor,
|
private val moduleDescriptor: ModuleDescriptor,
|
||||||
storageManager: StorageManager,
|
storageManager: StorageManager,
|
||||||
deferredOwnerModuleDescriptor: () -> ModuleDescriptor,
|
deferredOwnerModuleDescriptor: () -> ModuleDescriptor,
|
||||||
+2
-2
@@ -53,8 +53,8 @@ class DeserializationComponentsForJava(
|
|||||||
storageManager, moduleDescriptor, configuration, classDataFinder, annotationAndConstantLoader, packageFragmentProvider,
|
storageManager, moduleDescriptor, configuration, classDataFinder, annotationAndConstantLoader, packageFragmentProvider,
|
||||||
LocalClassifierTypeSettings.Default, errorReporter, lookupTracker, JavaFlexibleTypeDeserializer,
|
LocalClassifierTypeSettings.Default, errorReporter, lookupTracker, JavaFlexibleTypeDeserializer,
|
||||||
emptyList(), notFoundClasses, contractDeserializer,
|
emptyList(), notFoundClasses, contractDeserializer,
|
||||||
additionalClassPartsProvider = jvmBuiltIns?.settings ?: AdditionalClassPartsProvider.None,
|
additionalClassPartsProvider = jvmBuiltIns?.customizer ?: AdditionalClassPartsProvider.None,
|
||||||
platformDependentDeclarationFilter = jvmBuiltIns?.settings ?: PlatformDependentDeclarationFilter.NoPlatformDependent,
|
platformDependentDeclarationFilter = jvmBuiltIns?.customizer ?: PlatformDependentDeclarationFilter.NoPlatformDependent,
|
||||||
extensionRegistryLite = JvmProtoBufUtil.EXTENSION_REGISTRY,
|
extensionRegistryLite = JvmProtoBufUtil.EXTENSION_REGISTRY,
|
||||||
kotlinTypeChecker = kotlinTypeChecker, samConversionResolver = SamConversionResolverImpl(storageManager, emptyList())
|
kotlinTypeChecker = kotlinTypeChecker, samConversionResolver = SamConversionResolverImpl(storageManager, emptyList())
|
||||||
)
|
)
|
||||||
|
|||||||
+1
-1
@@ -85,7 +85,7 @@ class RuntimeModuleData private constructor(
|
|||||||
// .kotlin_builtins files should be found by the same class loader that loaded stdlib classes
|
// .kotlin_builtins files should be found by the same class loader that loaded stdlib classes
|
||||||
val stdlibClassLoader = Unit::class.java.classLoader
|
val stdlibClassLoader = Unit::class.java.classLoader
|
||||||
val builtinsProvider = JvmBuiltInsPackageFragmentProvider(
|
val builtinsProvider = JvmBuiltInsPackageFragmentProvider(
|
||||||
storageManager, ReflectKotlinClassFinder(stdlibClassLoader), module, notFoundClasses, builtIns.settings, builtIns.settings,
|
storageManager, ReflectKotlinClassFinder(stdlibClassLoader), module, notFoundClasses, builtIns.customizer, builtIns.customizer,
|
||||||
DeserializationConfiguration.Default, NewKotlinTypeChecker.Default, SamConversionResolverImpl(storageManager, emptyList())
|
DeserializationConfiguration.Default, NewKotlinTypeChecker.Default, SamConversionResolverImpl(storageManager, emptyList())
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user