commit c3fdcf6dde48e7052903a9aa07108217ac02d086
author Hector Oswaldo Caballero <hector.caballero@ericsson.com> Thu Sep 21 06:22:34 2017 -0400
committer Hector Oswaldo Caballero <hector.caballero@ericsson.com> Thu Sep 21 06:22:34 2017 -0400
tree 831ccd51333534b04796008d68a444dbe0bb6e3d
parent 5faf32b22ec5057bf4d883a14f6eea1212211a8e

Fix statement unnecessarily nested within else clause

Change-Id: I8fae6a7e6b1465608f9c5f20f3507d6d02cc608d

src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java b/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java
index 2770f00..ea2e556 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java
@@ -150,9 +150,8 @@
   private static boolean match(String value, String pattern, boolean refMatcher) {
     if (refMatcher) {
       return RefPatternMatcher.getMatcher(pattern).match(value, null);
-    } else {
-      return Pattern.matches(pattern, value);
     }
+    return Pattern.matches(pattern, value);
   }
 
   private boolean canSkipGroup(PluginConfig conf, IdentifiedUser user) {