Drop traces of old annotations from kotlin.jvm.internal
This commit is contained in:
@@ -557,7 +557,6 @@
|
|||||||
<withKotlin modulename="kotlin-compiler">
|
<withKotlin modulename="kotlin-compiler">
|
||||||
<compilerarg value="-version"/>
|
<compilerarg value="-version"/>
|
||||||
</withKotlin>
|
</withKotlin>
|
||||||
<skip pattern="kotlin/jvm/internal/.*"/>
|
|
||||||
<skip pattern="kotlin/Metadata"/>
|
<skip pattern="kotlin/Metadata"/>
|
||||||
<src refid="compilerSources.path"/>
|
<src refid="compilerSources.path"/>
|
||||||
<classpath refid="classpath"/>
|
<classpath refid="classpath"/>
|
||||||
@@ -617,7 +616,7 @@
|
|||||||
<withKotlin modulename="kotlin-build-common">
|
<withKotlin modulename="kotlin-build-common">
|
||||||
<compilerarg value="-version"/>
|
<compilerarg value="-version"/>
|
||||||
</withKotlin>
|
</withKotlin>
|
||||||
<skip pattern="kotlin/jvm/internal/.*"/>
|
<skip pattern="kotlin/Metadata"/>
|
||||||
<src>
|
<src>
|
||||||
<pathelement path="build-common/src"/>
|
<pathelement path="build-common/src"/>
|
||||||
</src>
|
</src>
|
||||||
@@ -692,7 +691,6 @@
|
|||||||
<withKotlin modulename="kotlin-android-compiler-plugin">
|
<withKotlin modulename="kotlin-android-compiler-plugin">
|
||||||
<compilerarg value="-version"/>
|
<compilerarg value="-version"/>
|
||||||
</withKotlin>
|
</withKotlin>
|
||||||
<skip pattern="kotlin/jvm/internal/.*"/>
|
|
||||||
<skip pattern="kotlin/Metadata"/>
|
<skip pattern="kotlin/Metadata"/>
|
||||||
<src>
|
<src>
|
||||||
<pathelement path="plugins/android-compiler-plugin/src"/>
|
<pathelement path="plugins/android-compiler-plugin/src"/>
|
||||||
@@ -719,7 +717,6 @@
|
|||||||
<withKotlin modulename="kotlin-ant-tools">
|
<withKotlin modulename="kotlin-ant-tools">
|
||||||
<compilerarg value="-version"/>
|
<compilerarg value="-version"/>
|
||||||
</withKotlin>
|
</withKotlin>
|
||||||
<skip pattern="kotlin/jvm/internal/.*"/>
|
|
||||||
<skip pattern="kotlin/Metadata"/>
|
<skip pattern="kotlin/Metadata"/>
|
||||||
<src>
|
<src>
|
||||||
<dirset dir="${basedir}/ant">
|
<dirset dir="${basedir}/ant">
|
||||||
@@ -799,7 +796,6 @@
|
|||||||
|
|
||||||
<javac2 srcdir="${toString:src.dirset}" destdir="@{output}" debug="true" debuglevel="lines,vars,source"
|
<javac2 srcdir="${toString:src.dirset}" destdir="@{output}" debug="true" debuglevel="lines,vars,source"
|
||||||
includeAntRuntime="false" source="${java.target}" target="${java.target}">
|
includeAntRuntime="false" source="${java.target}" target="${java.target}">
|
||||||
<skip pattern="kotlin/jvm/internal/.*"/>
|
|
||||||
<skip pattern="kotlin/Metadata"/>
|
<skip pattern="kotlin/Metadata"/>
|
||||||
<classpath>
|
<classpath>
|
||||||
<path refid="classpath.path"/>
|
<path refid="classpath.path"/>
|
||||||
@@ -975,7 +971,7 @@
|
|||||||
<kotlinc src="${basedir}/generators/infrastructure/strip-kotlin-annotations.kts" output="">
|
<kotlinc src="${basedir}/generators/infrastructure/strip-kotlin-annotations.kts" output="">
|
||||||
<compilerarg value="-version"/>
|
<compilerarg value="-version"/>
|
||||||
<compilerarg value="-script"/>
|
<compilerarg value="-script"/>
|
||||||
<compilerarg value="kotlin/jvm/internal/.*|kotlin/Metadata"/> <!-- Annotations to strip -->
|
<compilerarg value="kotlin/Metadata"/> <!-- Annotation to strip -->
|
||||||
<compilerarg value="kotlin/reflect/jvm/internal/impl/.*"/> <!-- Classes to strip from -->
|
<compilerarg value="kotlin/reflect/jvm/internal/impl/.*"/> <!-- Classes to strip from -->
|
||||||
<compilerarg value="${output}/kotlin-reflect-jarjar.jar"/>
|
<compilerarg value="${output}/kotlin-reflect-jarjar.jar"/>
|
||||||
<compilerarg value="${kotlin-home}/lib/kotlin-reflect.jar"/>
|
<compilerarg value="${kotlin-home}/lib/kotlin-reflect.jar"/>
|
||||||
@@ -1071,7 +1067,6 @@
|
|||||||
<withKotlin modulename="kotlin-for-upsource">
|
<withKotlin modulename="kotlin-for-upsource">
|
||||||
<compilerarg value="-version"/>
|
<compilerarg value="-version"/>
|
||||||
</withKotlin>
|
</withKotlin>
|
||||||
<skip pattern="kotlin/jvm/internal/.*"/>
|
|
||||||
<skip pattern="kotlin/Metadata"/>
|
<skip pattern="kotlin/Metadata"/>
|
||||||
<src>
|
<src>
|
||||||
<dirset dir="${basedir}/idea/ide-common" includes="src"/>
|
<dirset dir="${basedir}/idea/ide-common" includes="src"/>
|
||||||
|
|||||||
@@ -25,6 +25,7 @@ import org.jetbrains.kotlin.asJava.LightClassTestCommon
|
|||||||
import org.jetbrains.kotlin.idea.KotlinDaemonAnalyzerTestCase
|
import org.jetbrains.kotlin.idea.KotlinDaemonAnalyzerTestCase
|
||||||
import org.jetbrains.kotlin.idea.test.KotlinLightCodeInsightFixtureTestCase
|
import org.jetbrains.kotlin.idea.test.KotlinLightCodeInsightFixtureTestCase
|
||||||
import org.jetbrains.kotlin.idea.test.KotlinWithJdkAndRuntimeLightProjectDescriptor
|
import org.jetbrains.kotlin.idea.test.KotlinWithJdkAndRuntimeLightProjectDescriptor
|
||||||
|
import org.jetbrains.kotlin.load.java.JvmAnnotationNames
|
||||||
import org.jetbrains.kotlin.test.KotlinTestUtils
|
import org.jetbrains.kotlin.test.KotlinTestUtils
|
||||||
import org.jetbrains.kotlin.test.MockLibraryUtil
|
import org.jetbrains.kotlin.test.MockLibraryUtil
|
||||||
import org.junit.Assert
|
import org.junit.Assert
|
||||||
@@ -80,8 +81,7 @@ private fun testLightClass(project: Project, testDataPath: String, normalize: (S
|
|||||||
.replace("java.lang.String s)", "java.lang.String p)")
|
.replace("java.lang.String s)", "java.lang.String p)")
|
||||||
.replace("java.lang.String s1", "java.lang.String p1")
|
.replace("java.lang.String s1", "java.lang.String p1")
|
||||||
.replace("java.lang.String s2", "java.lang.String p2")
|
.replace("java.lang.String s2", "java.lang.String p2")
|
||||||
.removeLinesStartingWith("@kotlin.jvm.internal.")
|
.removeLinesStartingWith("@" + JvmAnnotationNames.METADATA_FQ_NAME.asString())
|
||||||
.removeLinesStartingWith("@kotlin.Metadata")
|
|
||||||
.run(normalize)
|
.run(normalize)
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
|
|||||||
+2
-1
@@ -23,6 +23,7 @@ import org.jetbrains.kotlin.codegen.DelegatingClassBuilder
|
|||||||
import org.jetbrains.kotlin.codegen.DelegatingClassBuilderFactory
|
import org.jetbrains.kotlin.codegen.DelegatingClassBuilderFactory
|
||||||
import org.jetbrains.kotlin.codegen.extensions.ClassBuilderInterceptorExtension
|
import org.jetbrains.kotlin.codegen.extensions.ClassBuilderInterceptorExtension
|
||||||
import org.jetbrains.kotlin.diagnostics.DiagnosticSink
|
import org.jetbrains.kotlin.diagnostics.DiagnosticSink
|
||||||
|
import org.jetbrains.kotlin.load.java.JvmAnnotationNames
|
||||||
import org.jetbrains.kotlin.resolve.BindingContext
|
import org.jetbrains.kotlin.resolve.BindingContext
|
||||||
import org.jetbrains.kotlin.resolve.jvm.diagnostics.JvmDeclarationOrigin
|
import org.jetbrains.kotlin.resolve.jvm.diagnostics.JvmDeclarationOrigin
|
||||||
import org.jetbrains.org.objectweb.asm.*
|
import org.jetbrains.org.objectweb.asm.*
|
||||||
@@ -165,7 +166,7 @@ abstract class AnnotationCollectorExtensionBase(val supportInheritedAnnotations:
|
|||||||
private fun isAnnotationHandled(annotationFqName: String): Boolean {
|
private fun isAnnotationHandled(annotationFqName: String): Boolean {
|
||||||
return if (annotationFilterEnabled)
|
return if (annotationFilterEnabled)
|
||||||
annotationFilters.any { it.matcher(annotationFqName).matches() }
|
annotationFilters.any { it.matcher(annotationFqName).matches() }
|
||||||
else !annotationFqName.startsWith("kotlin.jvm.internal.") && annotationFqName != "kotlin.Metadata" //apply to all
|
else annotationFqName != JvmAnnotationNames.METADATA_FQ_NAME.asString()
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun recordClass(packageName: String, className: String) {
|
private fun recordClass(packageName: String, className: String) {
|
||||||
|
|||||||
Reference in New Issue
Block a user