From b2efbf271fe261750b1d75676ed9fa0cce7e8db9 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Fri, 29 May 2015 16:11:46 +0900 Subject: [PATCH] HookTestCase: Use auto-closeable streams Change-Id: Icc9ad197e95ee982c119b18c197c095cd43f41fc --- .../google/gerrit/server/tools/hooks/HookTestCase.java | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/gerrit-server/src/test/java/com/google/gerrit/server/tools/hooks/HookTestCase.java b/gerrit-server/src/test/java/com/google/gerrit/server/tools/hooks/HookTestCase.java index 219b839c95..f915d05a97 100644 --- a/gerrit-server/src/test/java/com/google/gerrit/server/tools/hooks/HookTestCase.java +++ b/gerrit-server/src/test/java/com/google/gerrit/server/tools/hooks/HookTestCase.java @@ -122,18 +122,12 @@ public abstract class HookTestCase extends LocalDiskRepositoryTestCase { hook.setLastModified(time); hooks.put(name, hook); } else if ("jar".equals(protocol)) { - InputStream in = url.openStream(); - try { + try (InputStream in = url.openStream()) { hook = File.createTempFile("hook_", ".sh"); cleanup.add(hook); - FileOutputStream out = new FileOutputStream(hook); - try { + try (FileOutputStream out = new FileOutputStream(hook)) { ByteStreams.copy(in, out); - } finally { - out.close(); } - } finally { - in.close(); } hook.setExecutable(true); hooks.put(name, hook);