Skip to content

Commit e5e3dd2

Browse files
committed
code simplifications
1 parent e97ba77 commit e5e3dd2

File tree

2 files changed

+7
-19
lines changed
  • maven-plugin-tools-api/src/main/java/org/apache/maven/tools/plugin/util
  • maven-plugin-tools-generators/src/main/java/org/apache/maven/tools/plugin/generator

2 files changed

+7
-19
lines changed

maven-plugin-tools-api/src/main/java/org/apache/maven/tools/plugin/util/PluginUtils.java

+2-15
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
import java.net.URL;
2424
import java.net.URLClassLoader;
2525
import java.util.ArrayList;
26-
import java.util.Collections;
2726
import java.util.Comparator;
2827
import java.util.List;
2928

@@ -85,13 +84,7 @@ public static String[] findSources(String basedir, String include, String exclud
8584
*/
8685
public static void sortMojos(List<MojoDescriptor> mojoDescriptors) {
8786
if (mojoDescriptors != null) {
88-
Collections.sort(mojoDescriptors, new Comparator<MojoDescriptor>() {
89-
/** {@inheritDoc} */
90-
@Override
91-
public int compare(MojoDescriptor mojo0, MojoDescriptor mojo1) {
92-
return mojo0.getGoal().compareToIgnoreCase(mojo1.getGoal());
93-
}
94-
});
87+
mojoDescriptors.sort(Comparator.comparing(MojoDescriptor::getGoal));
9588
}
9689
}
9790

@@ -104,13 +97,7 @@ public int compare(MojoDescriptor mojo0, MojoDescriptor mojo1) {
10497
*/
10598
public static void sortMojoParameters(List<Parameter> parameters) {
10699
if (parameters != null) {
107-
Collections.sort(parameters, new Comparator<Parameter>() {
108-
/** {@inheritDoc} */
109-
@Override
110-
public int compare(Parameter parameter1, Parameter parameter2) {
111-
return parameter1.getName().compareToIgnoreCase(parameter2.getName());
112-
}
113-
});
100+
parameters.sort(Comparator.comparing(Parameter::getName));
114101
}
115102
}
116103

maven-plugin-tools-generators/src/main/java/org/apache/maven/tools/plugin/generator/GeneratorUtils.java

+5-4
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import java.util.LinkedList;
3434
import java.util.List;
3535
import java.util.Map;
36+
import java.util.Objects;
3637
import java.util.Stack;
3738
import java.util.regex.Matcher;
3839
import java.util.regex.Pattern;
@@ -178,9 +179,9 @@ static String decodeJavadocTags(String description) {
178179
while (matcher.find()) {
179180
String tag = matcher.group(1);
180181
String text = matcher.group(2);
181-
text = StringUtils.replace(text, "&", "&amp;");
182-
text = StringUtils.replace(text, "<", "&lt;");
183-
text = StringUtils.replace(text, ">", "&gt;");
182+
text = text.replace("&", "&amp;");
183+
text = text.replace("<", "&lt;");
184+
text = text.replace(">", "&gt;");
184185
if ("code".equals(tag)) {
185186
text = "<code>" + text + "</code>";
186187
} else if ("link".equals(tag) || "linkplain".equals(tag) || "value".equals(tag)) {
@@ -516,7 +517,7 @@ public static String discoverPackageName(PluginDescriptor pluginDescriptor) {
516517
for (MojoDescriptor descriptor : mojoDescriptors) {
517518

518519
String impl = descriptor.getImplementation();
519-
if (StringUtils.equals(descriptor.getGoal(), "help") && StringUtils.equals("HelpMojo", impl)) {
520+
if (Objects.equals(descriptor.getGoal(), "help") && Objects.equals("HelpMojo", impl)) {
520521
continue;
521522
}
522523
if (impl.lastIndexOf('.') != -1) {

0 commit comments

Comments
 (0)