From 50e7ff40975f069a117a0daa230723c5ef180b8c Mon Sep 17 00:00:00 2001 From: Mikhail Zarechenskiy Date: Wed, 14 Aug 2019 19:10:34 +0300 Subject: [PATCH] Refactoring: move `isSnapshot` function closer to similar ones --- .../configuration/GradleKotlinFrameworkSupportProvider.kt | 1 + .../KotlinDslGradleKotlinFrameworkSupportProvider.kt | 1 + .../idea/configuration/ConfigureKotlinInProjectUtils.kt | 2 +- idea/src/org/jetbrains/kotlin/idea/util/versioning.kt | 6 +++++- .../kotlin/idea/versions/KotlinRuntimeLibraryUtil.kt | 8 +------- 5 files changed, 9 insertions(+), 9 deletions(-) diff --git a/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/GradleKotlinFrameworkSupportProvider.kt b/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/GradleKotlinFrameworkSupportProvider.kt index 04b79fef52a..588e2083521 100644 --- a/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/GradleKotlinFrameworkSupportProvider.kt +++ b/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/GradleKotlinFrameworkSupportProvider.kt @@ -32,6 +32,7 @@ import org.jetbrains.kotlin.idea.formatter.KotlinStyleGuideCodeStyle import org.jetbrains.kotlin.idea.formatter.ProjectCodeStyleImporter import org.jetbrains.kotlin.idea.statistics.FUSEventGroups import org.jetbrains.kotlin.idea.statistics.KotlinFUSLogger +import org.jetbrains.kotlin.idea.util.isSnapshot import org.jetbrains.kotlin.idea.versions.* import org.jetbrains.plugins.gradle.frameworkSupport.BuildScriptDataBuilder import org.jetbrains.plugins.gradle.frameworkSupport.GradleFrameworkSupportProvider diff --git a/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/KotlinDslGradleKotlinFrameworkSupportProvider.kt b/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/KotlinDslGradleKotlinFrameworkSupportProvider.kt index e61f7fea05c..78a929f800b 100644 --- a/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/KotlinDslGradleKotlinFrameworkSupportProvider.kt +++ b/idea/idea-gradle/src/org/jetbrains/kotlin/idea/configuration/KotlinDslGradleKotlinFrameworkSupportProvider.kt @@ -32,6 +32,7 @@ import org.jetbrains.kotlin.idea.formatter.KotlinStyleGuideCodeStyle import org.jetbrains.kotlin.idea.formatter.ProjectCodeStyleImporter import org.jetbrains.kotlin.idea.statistics.FUSEventGroups import org.jetbrains.kotlin.idea.statistics.KotlinFUSLogger +import org.jetbrains.kotlin.idea.util.isSnapshot import org.jetbrains.kotlin.idea.versions.* import org.jetbrains.plugins.gradle.frameworkSupport.BuildScriptDataBuilder import org.jetbrains.plugins.gradle.frameworkSupport.KotlinDslGradleFrameworkSupportProvider diff --git a/idea/idea-jvm/src/org/jetbrains/kotlin/idea/configuration/ConfigureKotlinInProjectUtils.kt b/idea/idea-jvm/src/org/jetbrains/kotlin/idea/configuration/ConfigureKotlinInProjectUtils.kt index 34aec57cdf8..0cbf796db70 100644 --- a/idea/idea-jvm/src/org/jetbrains/kotlin/idea/configuration/ConfigureKotlinInProjectUtils.kt +++ b/idea/idea-jvm/src/org/jetbrains/kotlin/idea/configuration/ConfigureKotlinInProjectUtils.kt @@ -36,7 +36,7 @@ import org.jetbrains.kotlin.idea.util.projectStructure.sdk import org.jetbrains.kotlin.idea.util.projectStructure.version import org.jetbrains.kotlin.idea.versions.SuppressNotificationState import org.jetbrains.kotlin.idea.versions.hasKotlinJsKjsmFile -import org.jetbrains.kotlin.idea.versions.isSnapshot +import org.jetbrains.kotlin.idea.util.isSnapshot import org.jetbrains.kotlin.idea.vfilefinder.IDEVirtualFileFinder import org.jetbrains.kotlin.resolve.jvm.modules.KOTLIN_STDLIB_MODULE_NAME import org.jetbrains.kotlin.utils.ifEmpty diff --git a/idea/src/org/jetbrains/kotlin/idea/util/versioning.kt b/idea/src/org/jetbrains/kotlin/idea/util/versioning.kt index 6d6c40a8bf2..6e510513b1e 100644 --- a/idea/src/org/jetbrains/kotlin/idea/util/versioning.kt +++ b/idea/src/org/jetbrains/kotlin/idea/util/versioning.kt @@ -16,7 +16,11 @@ fun isDev(version: String): Boolean { return version.contains("dev") } +fun isSnapshot(version: String): Boolean { + return version.contains("SNAPSHOT", ignoreCase = true) +} + val buildNumber: String by lazy { val file = File(PathUtil.kotlinPathsForIdeaPlugin.homePath, "build.txt") if (file.exists()) file.readText().trim() else "unknown" -} \ No newline at end of file +} diff --git a/idea/src/org/jetbrains/kotlin/idea/versions/KotlinRuntimeLibraryUtil.kt b/idea/src/org/jetbrains/kotlin/idea/versions/KotlinRuntimeLibraryUtil.kt index 606e27515a3..367677e71a4 100644 --- a/idea/src/org/jetbrains/kotlin/idea/versions/KotlinRuntimeLibraryUtil.kt +++ b/idea/src/org/jetbrains/kotlin/idea/versions/KotlinRuntimeLibraryUtil.kt @@ -32,19 +32,17 @@ import com.intellij.openapi.ui.Messages import com.intellij.openapi.vfs.JarFileSystem import com.intellij.openapi.vfs.VfsUtil import com.intellij.openapi.vfs.VirtualFile -import com.intellij.psi.JavaPsiFacade -import com.intellij.psi.PsiClass import com.intellij.psi.search.GlobalSearchScope import com.intellij.util.containers.MultiMap import com.intellij.util.indexing.FileBasedIndex import com.intellij.util.indexing.ScalarIndexExtension import com.intellij.util.text.VersionComparatorUtil -import org.jetbrains.kotlin.builtins.KotlinBuiltIns import org.jetbrains.kotlin.config.JvmTarget import org.jetbrains.kotlin.config.KotlinCompilerVersion import org.jetbrains.kotlin.idea.framework.JavaRuntimeDetectionUtil import org.jetbrains.kotlin.idea.framework.isExternalLibrary import org.jetbrains.kotlin.idea.util.application.runReadAction +import org.jetbrains.kotlin.idea.util.isSnapshot import org.jetbrains.kotlin.idea.util.projectStructure.version import org.jetbrains.kotlin.idea.util.runWithAlternativeResolveEnabled import org.jetbrains.kotlin.idea.vfilefinder.KotlinJavaScriptMetaFileIndex @@ -235,10 +233,6 @@ fun getDefaultJvmTarget(sdk: Sdk?, version: String): JvmTarget? { } } -fun isSnapshot(version: String): Boolean { - return version.contains("SNAPSHOT", ignoreCase = true) -} - fun hasJdkLikeUpdatedRuntime(version: String): Boolean = VersionComparatorUtil.compare(version, "1.2.0-rc-39") >= 0 || isSnapshot(version) ||