Remove Unnecessary ImmutableList.Builder
Minor change to tidy up the regex validation code, creating an
immutable list in-place instead of using a builder.
Change-Id: Ia617af24a93f9d58a98a3601e687474500906d2c
diff --git a/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/PluginConfigValidator.java b/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/PluginConfigValidator.java
index 430e92b..d9cc872 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/PluginConfigValidator.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/PluginConfigValidator.java
@@ -151,14 +151,13 @@
public ImmutableList<CommitValidationMessage> validateRegex(
String fileName, Config cfg, String validatorKey) {
- ImmutableList.Builder<CommitValidationMessage> validationMessages = ImmutableList.builder();
String pattern = cfg.getString("plugin", pluginName, validatorKey);
if (pattern != null) {
try {
Pattern.compile(pattern);
} catch (PatternSyntaxException e) {
- validationMessages.add(
+ return ImmutableList.of(
new CommitValidationMessage(
String.format(
"The value '%s' configured in %s (parameter %s.%s) is invalid.",
@@ -166,7 +165,7 @@
ValidationMessage.Type.ERROR));
}
}
- return validationMessages.build();
+ return ImmutableList.of();
}
/**