Renaming remaining former "environment"-related identifiers

This commit is contained in:
Ilya Chernikov
2018-09-03 20:25:14 +02:00
parent 7860220e76
commit 836ea28fe7
12 changed files with 30 additions and 30 deletions
@@ -58,7 +58,7 @@ import kotlin.script.experimental.host.ScriptingHostConfiguration
import kotlin.script.experimental.host.configurationDependencies import kotlin.script.experimental.host.configurationDependencies
import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate
import kotlin.script.experimental.jvm.JvmDependency import kotlin.script.experimental.jvm.JvmDependency
import kotlin.script.experimental.jvm.defaultJvmScriptingEnvironment import kotlin.script.experimental.jvm.defaultJvmScriptingHostConfiguration
import kotlin.script.experimental.location.ScriptExpectedLocation import kotlin.script.experimental.location.ScriptExpectedLocation
import kotlin.script.templates.standard.ScriptTemplateWithArgs import kotlin.script.templates.standard.ScriptTemplateWithArgs
@@ -210,7 +210,7 @@ fun loadDefinitionsFromTemplates(
) )
} }
template.annotations.firstIsInstanceOrNull<kotlin.script.experimental.annotations.KotlinScript>() != null -> { template.annotations.firstIsInstanceOrNull<kotlin.script.experimental.annotations.KotlinScript>() != null -> {
val hostConfiguration = ScriptingHostConfiguration(defaultJvmScriptingEnvironment) { val hostConfiguration = ScriptingHostConfiguration(defaultJvmScriptingHostConfiguration) {
configurationDependencies(JvmDependency(classpath)) configurationDependencies(JvmDependency(classpath))
} }
KotlinScriptDefinitionAdapterFromNewAPI( KotlinScriptDefinitionAdapterFromNewAPI(
@@ -11,17 +11,17 @@ import kotlin.script.experimental.util.PropertiesCollection
interface ScriptEvaluationConfigurationKeys interface ScriptEvaluationConfigurationKeys
class ScriptEvaluationConfiguration(baseEvaluationEnvironments: Iterable<ScriptEvaluationConfiguration>, body: Builder.() -> Unit) : class ScriptEvaluationConfiguration(baseEvaluationConfigurations: Iterable<ScriptEvaluationConfiguration>, body: Builder.() -> Unit) :
PropertiesCollection(Builder(baseEvaluationEnvironments).apply(body).data) { PropertiesCollection(Builder(baseEvaluationConfigurations).apply(body).data) {
constructor(body: Builder.() -> Unit = {}) : this(emptyList(), body) constructor(body: Builder.() -> Unit = {}) : this(emptyList(), body)
constructor( constructor(
vararg baseConfigurations: ScriptEvaluationConfiguration, body: Builder.() -> Unit = {} vararg baseConfigurations: ScriptEvaluationConfiguration, body: Builder.() -> Unit = {}
) : this(baseConfigurations.asIterable(), body) ) : this(baseConfigurations.asIterable(), body)
class Builder internal constructor(baseEvaluationEnvironments: Iterable<ScriptEvaluationConfiguration>) : class Builder internal constructor(baseEvaluationConfigurations: Iterable<ScriptEvaluationConfiguration>) :
ScriptEvaluationConfigurationKeys, ScriptEvaluationConfigurationKeys,
PropertiesCollection.Builder(baseEvaluationEnvironments) PropertiesCollection.Builder(baseEvaluationConfigurations)
companion object : ScriptEvaluationConfigurationKeys companion object : ScriptEvaluationConfigurationKeys
} }
@@ -27,7 +27,7 @@ fun createCompilationConfigurationFromTemplate(
): ScriptCompilationConfiguration { ): ScriptCompilationConfiguration {
val getScriptingClass = hostConfiguration[ScriptingHostConfiguration.getScriptingClass] val getScriptingClass = hostConfiguration[ScriptingHostConfiguration.getScriptingClass]
?: throw IllegalArgumentException("${ERROR_MSG_PREFIX}Expecting 'getScriptingClass' parameter in the scripting environment") ?: throw IllegalArgumentException("${ERROR_MSG_PREFIX}Expecting 'getScriptingClass' parameter in the scripting host configuration")
val baseClass: KClass<*> = val baseClass: KClass<*> =
try { try {
@@ -12,17 +12,17 @@ import kotlin.script.experimental.util.PropertiesCollection
interface ScriptingHostConfigurationKeys interface ScriptingHostConfigurationKeys
class ScriptingHostConfiguration(baseScriptingEnvironments: Iterable<ScriptingHostConfiguration>, body: Builder.() -> Unit) : class ScriptingHostConfiguration(baseScriptingConfigurations: Iterable<ScriptingHostConfiguration>, body: Builder.() -> Unit) :
PropertiesCollection(Builder(baseScriptingEnvironments).apply(body).data) { PropertiesCollection(Builder(baseScriptingConfigurations).apply(body).data) {
constructor(body: Builder.() -> Unit = {}) : this(emptyList(), body) constructor(body: Builder.() -> Unit = {}) : this(emptyList(), body)
constructor( constructor(
vararg baseConfigurations: ScriptingHostConfiguration, body: Builder.() -> Unit = {} vararg baseConfigurations: ScriptingHostConfiguration, body: Builder.() -> Unit = {}
) : this(baseConfigurations.asIterable(), body) ) : this(baseConfigurations.asIterable(), body)
class Builder internal constructor(baseScriptingEnvironments: Iterable<ScriptingHostConfiguration>) : class Builder internal constructor(baseScriptingHostConfigurations: Iterable<ScriptingHostConfiguration>) :
ScriptingHostConfigurationKeys, ScriptingHostConfigurationKeys,
PropertiesCollection.Builder(baseScriptingEnvironments) PropertiesCollection.Builder(baseScriptingHostConfigurations)
companion object : ScriptingHostConfigurationKeys companion object : ScriptingHostConfigurationKeys
} }
@@ -11,17 +11,17 @@ import kotlin.script.experimental.api.ScriptEvaluator
import kotlin.script.experimental.host.ScriptingHostConfiguration import kotlin.script.experimental.host.ScriptingHostConfiguration
import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate
import kotlin.script.experimental.host.BasicScriptingHost import kotlin.script.experimental.host.BasicScriptingHost
import kotlin.script.experimental.jvm.defaultJvmScriptingEnvironment import kotlin.script.experimental.jvm.defaultJvmScriptingHostConfiguration
open class BasicJvmScriptingHost( open class BasicJvmScriptingHost(
hostConfiguration: ScriptingHostConfiguration = defaultJvmScriptingEnvironment, hostConfiguration: ScriptingHostConfiguration = defaultJvmScriptingHostConfiguration,
compiler: JvmScriptCompiler = JvmScriptCompiler(hostConfiguration), compiler: JvmScriptCompiler = JvmScriptCompiler(hostConfiguration),
evaluator: ScriptEvaluator = BasicJvmScriptEvaluator() evaluator: ScriptEvaluator = BasicJvmScriptEvaluator()
) : BasicScriptingHost(compiler, evaluator) ) : BasicScriptingHost(compiler, evaluator)
inline fun <reified T : Any> createCompilationConfigurationFromTemplate( inline fun <reified T : Any> createCompilationConfigurationFromTemplate(
hostConfiguration: ScriptingHostConfiguration = defaultJvmScriptingEnvironment, hostConfiguration: ScriptingHostConfiguration = defaultJvmScriptingHostConfiguration,
noinline body: ScriptCompilationConfiguration.Builder.() -> Unit = {} noinline body: ScriptCompilationConfiguration.Builder.() -> Unit = {}
): ScriptCompilationConfiguration = createCompilationConfigurationFromTemplate( ): ScriptCompilationConfiguration = createCompilationConfigurationFromTemplate(
KotlinType(T::class), KotlinType(T::class),
@@ -46,7 +46,7 @@ import kotlin.script.experimental.jvm.JvmDependency
import kotlin.script.experimental.jvm.impl.BridgeDependenciesResolver import kotlin.script.experimental.jvm.impl.BridgeDependenciesResolver
import kotlin.script.experimental.jvm.javaHome import kotlin.script.experimental.jvm.javaHome
import kotlin.script.experimental.jvm.jvm import kotlin.script.experimental.jvm.jvm
import kotlin.script.experimental.jvmhost.JvmScriptEvaluationEnvironment import kotlin.script.experimental.jvmhost.JvmScriptEvaluationConfiguration
import kotlin.script.experimental.jvmhost.KJvmCompilerProxy import kotlin.script.experimental.jvmhost.KJvmCompilerProxy
import kotlin.script.experimental.jvmhost.baseClassLoader import kotlin.script.experimental.jvmhost.baseClassLoader
import kotlin.script.experimental.util.getOrError import kotlin.script.experimental.util.getOrError
@@ -58,7 +58,7 @@ class KJvmCompiledScript<out ScriptBase : Any>(
) : CompiledScript<ScriptBase> { ) : CompiledScript<ScriptBase> {
override suspend fun getClass(scriptEvaluationConfiguration: ScriptEvaluationConfiguration?): ResultWithDiagnostics<KClass<*>> = try { override suspend fun getClass(scriptEvaluationConfiguration: ScriptEvaluationConfiguration?): ResultWithDiagnostics<KClass<*>> = try {
val baseClassLoader = scriptEvaluationConfiguration?.get(JvmScriptEvaluationEnvironment.baseClassLoader) val baseClassLoader = scriptEvaluationConfiguration?.get(JvmScriptEvaluationConfiguration.baseClassLoader)
?: Thread.currentThread().contextClassLoader ?: Thread.currentThread().contextClassLoader
val dependencies = compilationConfiguration[ScriptCompilationConfiguration.dependencies] val dependencies = compilationConfiguration[ScriptCompilationConfiguration.dependencies]
?.flatMap { (it as? JvmDependency)?.classpath?.map { it.toURI().toURL() } ?: emptyList() } ?.flatMap { (it as? JvmDependency)?.classpath?.map { it.toURI().toURL() } ?: emptyList() }
@@ -6,11 +6,11 @@
package kotlin.script.experimental.jvmhost.impl package kotlin.script.experimental.jvmhost.impl
import kotlin.script.experimental.host.ScriptingHostConfiguration import kotlin.script.experimental.host.ScriptingHostConfiguration
import kotlin.script.experimental.jvm.defaultJvmScriptingEnvironment import kotlin.script.experimental.jvm.defaultJvmScriptingHostConfiguration
fun ScriptingHostConfiguration.withDefaults(): ScriptingHostConfiguration = fun ScriptingHostConfiguration.withDefaults(): ScriptingHostConfiguration =
if (this == defaultJvmScriptingEnvironment) { if (this == defaultJvmScriptingHostConfiguration) {
this this
} else { } else {
ScriptingHostConfiguration(defaultJvmScriptingEnvironment, this) ScriptingHostConfiguration(defaultJvmScriptingHostConfiguration, this)
} }
@@ -9,7 +9,7 @@ package kotlin.script.experimental.jvmhost
import kotlin.script.experimental.api.* import kotlin.script.experimental.api.*
import kotlin.script.experimental.host.ScriptingHostConfiguration import kotlin.script.experimental.host.ScriptingHostConfiguration
import kotlin.script.experimental.jvm.defaultJvmScriptingEnvironment import kotlin.script.experimental.jvm.defaultJvmScriptingHostConfiguration
import kotlin.script.experimental.jvmhost.impl.KJvmCompilerImpl import kotlin.script.experimental.jvmhost.impl.KJvmCompilerImpl
import kotlin.script.experimental.jvmhost.impl.withDefaults import kotlin.script.experimental.jvmhost.impl.withDefaults
@@ -30,7 +30,7 @@ interface CompiledJvmScriptsCache {
} }
open class JvmScriptCompiler( open class JvmScriptCompiler(
hostConfiguration: ScriptingHostConfiguration = defaultJvmScriptingEnvironment, hostConfiguration: ScriptingHostConfiguration = defaultJvmScriptingHostConfiguration,
val compilerProxy: KJvmCompilerProxy = KJvmCompilerImpl(hostConfiguration.withDefaults()), val compilerProxy: KJvmCompilerProxy = KJvmCompilerImpl(hostConfiguration.withDefaults()),
val cache: CompiledJvmScriptsCache = CompiledJvmScriptsCache.NoCache val cache: CompiledJvmScriptsCache = CompiledJvmScriptsCache.NoCache
) : ScriptCompiler { ) : ScriptCompiler {
@@ -10,14 +10,14 @@ package kotlin.script.experimental.jvmhost
import kotlin.script.experimental.api.* import kotlin.script.experimental.api.*
import kotlin.script.experimental.util.PropertiesCollection import kotlin.script.experimental.util.PropertiesCollection
open class JvmScriptEvaluationEnvironment : PropertiesCollection.Builder() { open class JvmScriptEvaluationConfiguration : PropertiesCollection.Builder() {
companion object : JvmScriptEvaluationEnvironment() companion object : JvmScriptEvaluationConfiguration()
} }
val JvmScriptEvaluationEnvironment.baseClassLoader by PropertiesCollection.key<ClassLoader?>(Thread.currentThread().contextClassLoader) val JvmScriptEvaluationConfiguration.baseClassLoader by PropertiesCollection.key<ClassLoader?>(Thread.currentThread().contextClassLoader)
val ScriptEvaluationConfiguration.jvm get() = JvmScriptEvaluationEnvironment() val ScriptEvaluationConfiguration.jvm get() = JvmScriptEvaluationConfiguration()
open class BasicJvmScriptEvaluator : ScriptEvaluator { open class BasicJvmScriptEvaluator : ScriptEvaluator {
@@ -25,7 +25,7 @@ val JvmScriptingHostConfigurationKeys.javaHome by PropertiesCollection.key<File>
val ScriptingHostConfigurationKeys.jvm val ScriptingHostConfigurationKeys.jvm
get() = JvmScriptingHostConfigurationBuilder() get() = JvmScriptingHostConfigurationBuilder()
val defaultJvmScriptingEnvironment = ScriptingHostConfiguration { val defaultJvmScriptingHostConfiguration = ScriptingHostConfiguration {
getScriptingClass(JvmGetScriptingClass()) getScriptingClass(JvmGetScriptingClass())
} }
@@ -19,7 +19,7 @@ import kotlin.script.experimental.host.ScriptingHostConfiguration
import kotlin.script.experimental.host.configurationDependencies import kotlin.script.experimental.host.configurationDependencies
import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate
import kotlin.script.experimental.jvm.JvmDependency import kotlin.script.experimental.jvm.JvmDependency
import kotlin.script.experimental.jvm.defaultJvmScriptingEnvironment import kotlin.script.experimental.jvm.defaultJvmScriptingHostConfiguration
class LazyScriptDefinitionFromDiscoveredClass internal constructor( class LazyScriptDefinitionFromDiscoveredClass internal constructor(
private val annotationsFromAsm: ArrayList<BinAnnData>, private val annotationsFromAsm: ArrayList<BinAnnData>,
@@ -36,7 +36,7 @@ class LazyScriptDefinitionFromDiscoveredClass internal constructor(
) : this(loadAnnotationsFromClass(classBytes), className, classpath, messageCollector) ) : this(loadAnnotationsFromClass(classBytes), className, classpath, messageCollector)
override val hostConfiguration: ScriptingHostConfiguration by lazy(LazyThreadSafetyMode.PUBLICATION) { override val hostConfiguration: ScriptingHostConfiguration by lazy(LazyThreadSafetyMode.PUBLICATION) {
ScriptingHostConfiguration(defaultJvmScriptingEnvironment) { ScriptingHostConfiguration(defaultJvmScriptingHostConfiguration) {
configurationDependencies.append(JvmDependency(classpath)) configurationDependencies.append(JvmDependency(classpath))
} }
} }
@@ -19,7 +19,7 @@ import kotlin.coroutines.experimental.buildSequence
import kotlin.script.experimental.annotations.KotlinScript import kotlin.script.experimental.annotations.KotlinScript
import kotlin.script.experimental.api.KotlinType import kotlin.script.experimental.api.KotlinType
import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate import kotlin.script.experimental.host.createCompilationConfigurationFromTemplate
import kotlin.script.experimental.jvm.defaultJvmScriptingEnvironment import kotlin.script.experimental.jvm.defaultJvmScriptingHostConfiguration
import kotlin.script.templates.ScriptTemplateDefinition import kotlin.script.templates.ScriptTemplateDefinition
internal const val SCRIPT_DEFINITION_MARKERS_PATH = "META-INF/kotlin/script/templates/" internal const val SCRIPT_DEFINITION_MARKERS_PATH = "META-INF/kotlin/script/templates/"
@@ -272,7 +272,7 @@ private fun loadScriptDefinition(
val cls = classLoader.loadClass(template) val cls = classLoader.loadClass(template)
val def = val def =
if (cls.annotations.firstIsInstanceOrNull<KotlinScript>() != null) { if (cls.annotations.firstIsInstanceOrNull<KotlinScript>() != null) {
val environment = defaultJvmScriptingEnvironment val environment = defaultJvmScriptingHostConfiguration
KotlinScriptDefinitionAdapterFromNewAPI( KotlinScriptDefinitionAdapterFromNewAPI(
createCompilationConfigurationFromTemplate( createCompilationConfigurationFromTemplate(
KotlinType(cls.kotlin), KotlinType(cls.kotlin),