refactor: Add missing @Override to overriding and implementing methods

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.MissingOverrideAnnotation?organizationId=T3BlblJld3JpdGU%3D Co-authored-by: Moderne <[email protected]>
openrewrite · Mar 12, 2024 · 892b93c · 892b93c
1 parent 2bdf2c6
commit 892b93c
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/src/main/java/org/openrewrite/kotlin/KotlinParser.java b/src/main/java/org/openrewrite/kotlin/KotlinParser.java
@@ -329,6 +329,7 @@ private Collection<Path> resolvedClasspath() {
             return classpath;
         }
 
+        @Override
         public KotlinParser build() {
             return new KotlinParser(resolvedClasspath(), styles, logCompilationWarningsAndErrors, typeCache, moduleName, languageLevel, isKotlinScript);
         }
@@ -338,6 +339,7 @@ public String getDslName() {
             return "kotlin";
         }
 
+        @Override
         public KotlinParser.Builder clone() {
             KotlinParser.Builder clone = (KotlinParser.Builder) super.clone();
             clone.typeCache = this.typeCache.clone();

diff --git a/src/main/java/org/openrewrite/kotlin/KotlinTemplate.java b/src/main/java/org/openrewrite/kotlin/KotlinTemplate.java
@@ -82,6 +82,7 @@ public static class Builder extends JavaTemplate.Builder {
             this.code = code;
         }
 
+        @Override
         public Builder imports(String... fullyQualifiedTypeNames) {
             for (String typeName : fullyQualifiedTypeNames) {
                 validateImport(typeName);
@@ -105,16 +106,19 @@ Builder parser(KotlinParser.Builder parser) {
             return this;
         }
 
+        @Override
         public Builder doAfterVariableSubstitution(Consumer<String> afterVariableSubstitution) {
             this.onAfterVariableSubstitution = afterVariableSubstitution;
             return this;
         }
 
+        @Override
         public Builder doBeforeParseTemplate(Consumer<String> beforeParseTemplate) {
             this.onBeforeParseTemplate = beforeParseTemplate;
             return this;
         }
 
+        @Override
         public KotlinTemplate build() {
             return new KotlinTemplate(false, parser, code, imports,
                     onAfterVariableSubstitution, onBeforeParseTemplate);

diff --git a/src/main/java/org/openrewrite/kotlin/tree/K.java b/src/main/java/org/openrewrite/kotlin/tree/K.java
@@ -70,8 +70,10 @@ default <P> J acceptKotlin(KotlinVisitor<P> v, P p) {
         return v.defaultValue(this, p);
     }
 
+    @Override
     Space getPrefix();
 
+    @Override
     default List<Comment> getComments() {
         return getPrefix().getComments();
     }
@@ -191,20 +193,24 @@ public SourceFile withCharset(Charset charset) {
         JRightPadded<Package> packageDeclaration;
 
         @Nullable
+        @Override
         public Package getPackageDeclaration() {
             return packageDeclaration == null ? null : packageDeclaration.getElement();
         }
 
+        @Override
         public K.CompilationUnit withPackageDeclaration(Package packageDeclaration) {
             return getPadding().withPackageDeclaration(JRightPadded.withElement(this.packageDeclaration, packageDeclaration));
         }
 
         List<JRightPadded<Import>> imports;
 
+        @Override
         public List<Import> getImports() {
             return JRightPadded.getElements(imports);
         }
 
+        @Override
         public K.CompilationUnit withImports(List<Import> imports) {
             return getPadding().withImports(JRightPadded.withElements(this.imports, imports));
         }
@@ -223,6 +229,7 @@ public K.CompilationUnit withStatements(List<Statement> statements) {
         @Getter
         Space eof;
 
+        @Override
         @Transient
         public List<J.ClassDeclaration> getClasses() {
             return statements.stream()
@@ -242,6 +249,7 @@ public K.CompilationUnit withClasses(List<J.ClassDeclaration> classes) {
             return getPadding().withClasses(JRightPadded.withElements(this.getPadding().getClasses(), classes));
         }
 
+        @Override
         public <P> J acceptKotlin(KotlinVisitor<P> v, P p) {
             return v.visitCompilationUnit(this, p);
         }
@@ -251,6 +259,7 @@ public <P> TreeVisitor<?, PrintOutputCapture<P>> printer(Cursor cursor) {
             return new KotlinPrinter<>();
         }
 
+        @Override
         @Transient
         public TypesInUse getTypesInUse() {
             TypesInUse cache;
@@ -267,6 +276,7 @@ public TypesInUse getTypesInUse() {
             return cache;
         }
 
+        @Override
         public Padding getPadding() {
             Padding p;
             if (this.padding == null) {
@@ -1104,6 +1114,7 @@ final class FunctionType implements K, TypeTree, Expression {
         @With
         Space prefix;
 
+        @Override
         public Space getPrefix() {
             //noinspection ConstantConditions
             return prefix == null ? returnType.getElement().getPrefix() : prefix;
@@ -1112,6 +1123,7 @@ public Space getPrefix() {
         @With
         Markers markers;
 
+        @Override
         public Markers getMarkers() {
             // For backwards compatibility with older LST before there was a prefix field
             //noinspection ConstantConditions
@@ -1168,6 +1180,7 @@ public FunctionType withParameters(List<TypeTree> parameters) {
             return returnType != null && returnType.getElement() != null ? returnType.getElement().getType() : null;
         }
 
+        @Override
         public <T extends J> T withType(@Nullable JavaType type) {
             TypeTree newType = returnType.getElement().withType(type);
             //noinspection unchecked