commit 55292feea87b020e20eecf97e4724fdd7cc1283b
author Han-Wen Nienhuys <hanwen@google.com> Thu Nov 30 14:34:41 2017 +0100
committer Han-Wen Nienhuys <hanwen@google.com> Thu Nov 30 14:50:58 2017 +0100
tree 894a041bcfae171ac42067d1c99ac3c45f8bdde1
parent 07e5ad1479532938dbde3f07650f38d235b032fb

Don't catch never thrown exceptions

Change Iff1534ea removed some exceptions.

Tested:
  bazel build plugins/reviewers/...

Change-Id: Iacd2d4e12d244892f2e9579b94789eac5487aaba

src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java
index d4ca423..66daf79 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/reviewers/ChangeEventListener.java
@@ -19,7 +19,6 @@
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Sets;
-import com.google.gerrit.common.errors.NoSuchGroupException;
 import com.google.gerrit.extensions.annotations.PluginName;
 import com.google.gerrit.extensions.common.ChangeInfo;
 import com.google.gerrit.extensions.events.RevisionCreatedListener;
@@ -239,11 +238,11 @@
       try {
         reviewers.addAll(
             groupMembers.listAccounts(groupsCollection.get().parse(r).getGroupUUID(), p));
-      } catch (UnprocessableEntityException | NoSuchGroupException e) {
+      } catch (UnprocessableEntityException e) {
         log.warn(String.format("Reviewer %s is neither an account nor a group", r));
       } catch (NoSuchProjectException e) {
         log.warn(String.format("Failed to list accounts for group %s and project %s", r, p));
-      } catch (IOException | OrmException e) {
+      } catch (IOException e) {
         log.warn(String.format("Failed to list accounts for group %s", r), e);
       }
     }