commit b504566b78b7d5869d2c3d6dc4e516e777fe34d6
author Hector Oswaldo Caballero <hector.caballero@ericsson.com> Wed Jun 20 06:18:05 2018 -0400
committer Hector Oswaldo Caballero <hector.caballero@ericsson.com> Wed Jun 20 06:18:05 2018 -0400
tree 7a02c539f2f7304726ecb72383c556f6ea9d721e
parent 534cd3254c7e43648d36043b0d3f48ab4ae34a27

[HookTask] Close readers in try-with-resources

Change-Id: Idd7318d8422b89bb4b8b58e48fb507dc88830539

src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java b/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java
index 7fbf1a7..76de6e3 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/hooks/HookTask.java
@@ -126,12 +126,10 @@
       }
 
       if (log.isDebugEnabled()) {
-        BufferedReader br = new BufferedReader(new StringReader(result.getOutput()));
-        try {
-          String line;
-          while ((line = br.readLine()) != null) {
-            log.debug("hook[{}] output: {}", name, line);
-          }
+        try (BufferedReader br = new BufferedReader(new StringReader(result.getOutput()))) {
+          br.lines()
+              .filter(s -> !s.isEmpty())
+              .forEach(line -> log.debug("hook[{}] output: {}", name, line));
         } catch (IOException iox) {
           log.error("Error writing hook [{}] output", name, iox);
         }