[HookTask] Close readers in try-with-resources Change-Id: Idd7318d8422b89bb4b8b58e48fb507dc88830539
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); }