commit | c742429c70b8b858f6004c1314d1e5d92fefb2ec | [log] [tgz] |
---|---|---|
author | Christian Aistleitner <christian@quelltextlich.at> | Sat Jun 06 18:16:35 2015 +0200 |
committer | Christian Aistleitner <christian@quelltextlich.at> | Sat Jun 06 18:34:17 2015 +0200 |
tree | fed2b4f2ccbbef1fa4e0542c84fb0eabf4486c32 | |
parent | 754dcd5e664e8ff3d1f006442d42711b57bb1fb9 [diff] |
Get rid of unneeded nested catch blocks Since we already have an outer catch block, and both the inner and outer catch block do the same thing, we drop the inner catch block, and handle the exceptions at the outer block. Thereby, we simplify the catch logic. Change-Id: I5a9d8a576e4222f1dc27b4583f444aba7db80766
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewersbyblame/ChangeUpdatedListener.java b/src/main/java/com/googlesource/gerrit/plugins/reviewersbyblame/ChangeUpdatedListener.java index e395747..c352eca 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/reviewersbyblame/ChangeUpdatedListener.java +++ b/src/main/java/com/googlesource/gerrit/plugins/reviewersbyblame/ChangeUpdatedListener.java
@@ -175,14 +175,10 @@ } } }); - } catch (OrmException x) { - log.error(x.getMessage(), x); - } catch (IOException x) { - log.error(x.getMessage(), x); } finally { reviewDb.close(); } - } catch (OrmException x) { + } catch (OrmException|IOException x) { log.error(x.getMessage(), x); } finally { rw.close();