[MNG-7416] Simplify Boolean expressions and returns (#227)

Use this link to re-run the recipe: https://app.moderne.io/recipes/builder/SEvWu02zw?organizationId=QXBhY2hlIE1hdmVu Co-authored-by: Moderne <team@moderne.io>
apache · Sep 26, 2023 · b7138b0 · b7138b0
1 parent 247bff7
commit b7138b0
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 10 deletions.
diff --git a/...apache/maven/tools/plugin/extractor/annotations/datamodel/ParameterAnnotationContent.java b/...apache/maven/tools/plugin/extractor/annotations/datamodel/ParameterAnnotationContent.java
@@ -208,11 +208,7 @@ public boolean equals(Object o) {
         if (!Objects.equals(defaultValue, that.defaultValue)) {
             return false;
         }
-        if (!Objects.equals(property, that.property)) {
-            return false;
-        }
-
-        return true;
+        return Objects.equals(property, that.property);
     }
 
     @Override

diff --git a/...rg/apache/maven/tools/plugin/extractor/annotations/scanner/visitors/MojoClassVisitor.java b/...rg/apache/maven/tools/plugin/extractor/annotations/scanner/visitors/MojoClassVisitor.java
@@ -86,7 +86,7 @@ public List<MojoFieldVisitor> findFieldWithAnnotation(Class<?> annotation) {
     }
 
     public List<MojoFieldVisitor> findFieldWithAnnotation(Set<String> annotationClassNames) {
-        List<MojoFieldVisitor> mojoFieldVisitors = new ArrayList<MojoFieldVisitor>();
+        List<MojoFieldVisitor> mojoFieldVisitors = new ArrayList<>();
 
         for (MojoFieldVisitor mojoFieldVisitor : this.fieldVisitors) {
             Map<String, MojoAnnotationVisitor> filedVisitorMap = mojoFieldVisitor.getAnnotationVisitorMap();

diff --git a/...a/org/apache/maven/tools/plugin/extractor/javadoc/JavaJavadocMojoDescriptorExtractor.java b/...a/org/apache/maven/tools/plugin/extractor/javadoc/JavaJavadocMojoDescriptorExtractor.java
@@ -510,7 +510,7 @@ private Map<String, JavaField> extractFieldParameterTags(JavaClass javaClass) {
         if (superClass != null) {
             rawParams = extractFieldParameterTags(superClass);
         } else {
-            rawParams = new TreeMap<String, JavaField>();
+            rawParams = new TreeMap<>();
         }
 
         for (JavaField field : javaClass.getFields()) {

diff --git a/...java/org/apache/maven/tools/plugin/extractor/javadoc/JavaMojoDescriptorExtractorTest.java b/...java/org/apache/maven/tools/plugin/extractor/javadoc/JavaMojoDescriptorExtractorTest.java
@@ -35,7 +35,6 @@
 import org.apache.maven.tools.plugin.generator.Generator;
 import org.apache.maven.tools.plugin.generator.PluginDescriptorFilesGenerator;
 import org.apache.maven.tools.plugin.util.PluginUtils;
-import org.codehaus.plexus.component.repository.ComponentDependency;
 import org.codehaus.plexus.logging.Logger;
 import org.codehaus.plexus.logging.console.ConsoleLogger;
 import org.codehaus.plexus.util.FileUtils;
@@ -87,7 +86,7 @@ private PluginToolsRequest createRequest(String directory) {
 
         PluginDescriptor pluginDescriptor = new PluginDescriptor();
         pluginDescriptor.setGoalPrefix("test");
-        pluginDescriptor.setDependencies(new ArrayList<ComponentDependency>());
+        pluginDescriptor.setDependencies(new ArrayList<>());
 
         return new DefaultPluginToolsRequest(project, pluginDescriptor).setEncoding("UTF-8");
     }

diff --git a/...-script/maven-script-ant/src/main/java/org/apache/maven/script/ant/AntPropertyHelper.java b/...-script/maven-script-ant/src/main/java/org/apache/maven/script/ant/AntPropertyHelper.java
@@ -44,7 +44,7 @@ public class AntPropertyHelper extends PropertyHelper {
     private Log log;
     private ExpressionEvaluator exprEvaluator;
     private MavenProject mavenProject;
-    private Map<String, String> artifactMap = new HashMap<String, String>();
+    private Map<String, String> artifactMap = new HashMap<>();
 
     /**
      * @deprecated use the other constructor