Prohibit using DeprecatedSinceKotlin annotation without arguments

This commit is contained in:
Mikhail Zarechenskiy
2020-06-18 11:36:38 +03:00
parent 67100d5ebe
commit 790433984b
5 changed files with 19 additions and 3 deletions
@@ -103,6 +103,7 @@ public interface Errors {
DiagnosticFactory0<PsiElement> DEPRECATED_SINCE_KOTLIN_WITHOUT_DEPRECATED = DiagnosticFactory0.create(ERROR);
DiagnosticFactory0<PsiElement> DEPRECATED_SINCE_KOTLIN_WITH_DEPRECATED_LEVEL = DiagnosticFactory0.create(ERROR);
DiagnosticFactory0<PsiElement> DEPRECATED_SINCE_KOTLIN_WITH_UNORDERED_VERSIONS = DiagnosticFactory0.create(ERROR);
DiagnosticFactory0<PsiElement> DEPRECATED_SINCE_KOTLIN_WITHOUT_ARGUMENTS = DiagnosticFactory0.create(ERROR);
DiagnosticFactory2<PsiElement, String, String> API_NOT_AVAILABLE = DiagnosticFactory2.create(ERROR);
@@ -380,6 +380,7 @@ public class DefaultErrorMessages {
MAP.put(DEPRECATED_SINCE_KOTLIN_WITHOUT_DEPRECATED, "DeprecatedSinceKotlin annotation can be used only together with Deprecated annotation");
MAP.put(DEPRECATED_SINCE_KOTLIN_WITH_DEPRECATED_LEVEL, "DeprecatedSinceKotlin annotation can be used only with unspecified deprecation level of Deprecated annotation");
MAP.put(DEPRECATED_SINCE_KOTLIN_WITH_UNORDERED_VERSIONS, "Values of DeprecatedSinceKotlin annotation should be ordered so 'warningSince' <= 'errorSince' <= 'hiddenSince' if specified");
MAP.put(DEPRECATED_SINCE_KOTLIN_WITHOUT_ARGUMENTS, "DeprecatedSinceKotlin annotation should have at least one argument");
MAP.put(API_NOT_AVAILABLE, "This declaration is only available since Kotlin {0} and cannot be used with the specified API version {1}", STRING, STRING);
@@ -48,6 +48,15 @@ object DeprecatedSinceKotlinAnnotationChecker : DeclarationChecker {
return
}
if (deprecatedSinceAnnotation.allValueArguments.isEmpty()) {
context.trace.report(
Errors.DEPRECATED_SINCE_KOTLIN_WITHOUT_ARGUMENTS.on(
deprecatedSinceAnnotationName
)
)
return
}
fun AnnotationDescriptor.getCheckedSinceVersion(name: String) =
getSinceVersion(name).also { checkVersion(it, name, context, deprecatedSinceAnnotationName) }
@@ -0,0 +1,7 @@
@Deprecated("")
@DeprecatedSinceKotlin
fun foo() {}
fun test() {
foo()
}
@@ -1,7 +1,5 @@
// FIR_IDENTICAL
@Deprecated("")
@DeprecatedSinceKotlin
@<!DEPRECATED_SINCE_KOTLIN_WITHOUT_ARGUMENTS!>DeprecatedSinceKotlin<!>
fun foo() {}
fun test() {