Skip to content

Commit b7138b0

Browse files
[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 <[email protected]>
1 parent 247bff7 commit b7138b0

File tree

5 files changed

+5
-10
lines changed

5 files changed

+5
-10
lines changed

maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/extractor/annotations/datamodel/ParameterAnnotationContent.java

+1-5
Original file line numberDiff line numberDiff line change
@@ -208,11 +208,7 @@ public boolean equals(Object o) {
208208
if (!Objects.equals(defaultValue, that.defaultValue)) {
209209
return false;
210210
}
211-
if (!Objects.equals(property, that.property)) {
212-
return false;
213-
}
214-
215-
return true;
211+
return Objects.equals(property, that.property);
216212
}
217213

218214
@Override

maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/extractor/annotations/scanner/visitors/MojoClassVisitor.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ public List<MojoFieldVisitor> findFieldWithAnnotation(Class<?> annotation) {
8686
}
8787

8888
public List<MojoFieldVisitor> findFieldWithAnnotation(Set<String> annotationClassNames) {
89-
List<MojoFieldVisitor> mojoFieldVisitors = new ArrayList<MojoFieldVisitor>();
89+
List<MojoFieldVisitor> mojoFieldVisitors = new ArrayList<>();
9090

9191
for (MojoFieldVisitor mojoFieldVisitor : this.fieldVisitors) {
9292
Map<String, MojoAnnotationVisitor> filedVisitorMap = mojoFieldVisitor.getAnnotationVisitorMap();

maven-plugin-tools-java/src/main/java/org/apache/maven/tools/plugin/extractor/javadoc/JavaJavadocMojoDescriptorExtractor.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ private Map<String, JavaField> extractFieldParameterTags(JavaClass javaClass) {
510510
if (superClass != null) {
511511
rawParams = extractFieldParameterTags(superClass);
512512
} else {
513-
rawParams = new TreeMap<String, JavaField>();
513+
rawParams = new TreeMap<>();
514514
}
515515

516516
for (JavaField field : javaClass.getFields()) {

maven-plugin-tools-java/src/test/java/org/apache/maven/tools/plugin/extractor/javadoc/JavaMojoDescriptorExtractorTest.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@
3535
import org.apache.maven.tools.plugin.generator.Generator;
3636
import org.apache.maven.tools.plugin.generator.PluginDescriptorFilesGenerator;
3737
import org.apache.maven.tools.plugin.util.PluginUtils;
38-
import org.codehaus.plexus.component.repository.ComponentDependency;
3938
import org.codehaus.plexus.logging.Logger;
4039
import org.codehaus.plexus.logging.console.ConsoleLogger;
4140
import org.codehaus.plexus.util.FileUtils;
@@ -87,7 +86,7 @@ private PluginToolsRequest createRequest(String directory) {
8786

8887
PluginDescriptor pluginDescriptor = new PluginDescriptor();
8988
pluginDescriptor.setGoalPrefix("test");
90-
pluginDescriptor.setDependencies(new ArrayList<ComponentDependency>());
89+
pluginDescriptor.setDependencies(new ArrayList<>());
9190

9291
return new DefaultPluginToolsRequest(project, pluginDescriptor).setEncoding("UTF-8");
9392
}

maven-script/maven-script-ant/src/main/java/org/apache/maven/script/ant/AntPropertyHelper.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public class AntPropertyHelper extends PropertyHelper {
4444
private Log log;
4545
private ExpressionEvaluator exprEvaluator;
4646
private MavenProject mavenProject;
47-
private Map<String, String> artifactMap = new HashMap<String, String>();
47+
private Map<String, String> artifactMap = new HashMap<>();
4848

4949
/**
5050
* @deprecated use the other constructor

0 commit comments

Comments
 (0)