commit 5680d0321c899cd9d52b600632d8405b6dd3982e
author David Pursehouse <david.pursehouse@sonymobile.com> Mon Nov 30 15:08:37 2015 +0900
committer David Pursehouse <david.pursehouse@sonymobile.com> Mon Nov 30 15:08:37 2015 +0900
tree f3f2f5fcd0d2fe948ba0ffcdc110356d73fc6b59
parent 2807b5fdfff3acc323c52002dab473933bfd47e6

CommitMessageFetcher: Open Repository and RevWalk in try-with-resource

Change-Id: Iba40d056a45cc5a61c168b160c3abe98f46a09a4

src/main/java/com/googlesource/gerrit/plugins/hooks/util/CommitMessageFetcher.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/hooks/util/CommitMessageFetcher.java b/src/main/java/com/googlesource/gerrit/plugins/hooks/util/CommitMessageFetcher.java
index a04b175..83b33ce 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/hooks/util/CommitMessageFetcher.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/hooks/util/CommitMessageFetcher.java
@@ -25,15 +25,11 @@
   }
 
   public String fetch(String projectName, String commitId) throws IOException {
-    final Repository repo =
-        repoManager.openRepository(new NameKey(projectName));
-    try {
-      RevWalk revWalk = new RevWalk(repo);
+    try (Repository repo = repoManager.openRepository(new NameKey(projectName));
+        RevWalk revWalk = new RevWalk(repo)){
       RevCommit commit = revWalk.parseCommit(ObjectId.fromString(commitId));
 
       return commit.getFullMessage();
-    } finally {
-      repo.close();
     }
   }