Mark few JVM-compatibility declarations deprecated. #2029
Conversation
| @@ -11,12 +11,15 @@ package kotlin | |||
|
|
|||
| @Target(AnnotationTarget.PROPERTY, AnnotationTarget.FIELD) | |||
| @Retention(AnnotationRetention.SOURCE) | |||
| @Deprecated("@Volatile is meaningless in Native", level = DeprecationLevel.WARNING) | |||
There was a problem hiding this comment.
Btw, we already have these annotations in common stdlib with @OptionalExpectation:
https://github.com/JetBrains/kotlin/blob/master/libraries/stdlib/common/src/kotlin/JvmAnnotationsH.kt#L113
This is supposed to make these declarations redundant: common code with such an annotation would be properly compiled with Native backend. And these declarations can only be required to compile Kotlin/JVM code (which has default kotlin.jvm.* import) with Kotlin/Native, which doesn't seem to be a case we'd like to support, in light of Kotlin Multiplatform.
There was a problem hiding this comment.
If you don't pursue the full deprecation cycle, you can raise the deprecation level straight to ERROR for this annotation.
|
Consider unifying wording with the similar deprecations in JS stdlib: JetBrains/kotlin@50f75ad |
No description provided.