Avoid redundant array creation Arrays.asList accepts multiple arguments (varargs) so no need to create an array object to pass it as parameter. Change-Id: Ibeffdb54f61db04e1ccf023bb6b3c423ce405d18
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java index 85cbfb3..37f1851 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java +++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java
@@ -28,7 +28,7 @@ super( "Enable " + pluginName + " integration", "false", - Arrays.asList(new String[] {"false", "true", "enforced"}), + Arrays.asList("false", "true", "enforced"), true); this.pluginName = pluginName; this.pluginCfgFactory = pluginCfgFactory;
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java index 267a4b8..de4acd9 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java +++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java
@@ -35,7 +35,7 @@ public void addRelatedLink(String issueId, URL relatedUrl, String description) throws IOException { if (log.isDebugEnabled()) { - log.debug("addRelatedLink({},{},{})", new Object[] {issueId, relatedUrl, description}); + log.debug("addRelatedLink({},{},{})", issueId, relatedUrl, description); } }