diff options
author | David Pursehouse <david.pursehouse@gmail.com> | 2018-03-05 20:23:03 +0900 |
---|---|---|
committer | David Pursehouse <david.pursehouse@gmail.com> | 2018-03-05 20:23:03 +0900 |
commit | 594722530bd9b3fd4f073a12775d88f27ad62de0 (patch) | |
tree | 7104d42f5963edb08e0087c7f2b40abfb1b689dc /org.eclipse.jgit.pgm.test/tst | |
parent | 9e4a455f9bf24f59b7193639a28c8338ae256c87 (diff) | |
download | jgit-594722530bd9b3fd4f073a12775d88f27ad62de0.tar.gz jgit-594722530bd9b3fd4f073a12775d88f27ad62de0.zip |
ArchiveTest: Open auto-closeable resources in try-with-resource
Change-Id: If11017f21027b46c7a66e52e4bc0cc73f4fbdc07
Signed-off-by: David Pursehouse <david.pursehouse@gmail.com>
Diffstat (limited to 'org.eclipse.jgit.pgm.test/tst')
-rw-r--r-- | org.eclipse.jgit.pgm.test/tst/org/eclipse/jgit/pgm/ArchiveTest.java | 79 |
1 files changed, 36 insertions, 43 deletions
diff --git a/org.eclipse.jgit.pgm.test/tst/org/eclipse/jgit/pgm/ArchiveTest.java b/org.eclipse.jgit.pgm.test/tst/org/eclipse/jgit/pgm/ArchiveTest.java index 3b352ce2aa..5817566dca 100644 --- a/org.eclipse.jgit.pgm.test/tst/org/eclipse/jgit/pgm/ArchiveTest.java +++ b/org.eclipse.jgit.pgm.test/tst/org/eclipse/jgit/pgm/ArchiveTest.java @@ -633,9 +633,8 @@ public class ArchiveTest extends CLIRepositoryTestCase { } private void assertMagic(long offset, byte[] magicBytes, File file) throws Exception { - BufferedInputStream in = new BufferedInputStream( - new FileInputStream(file)); - try { + try (BufferedInputStream in = new BufferedInputStream( + new FileInputStream(file))) { if (offset > 0) { long skipped = in.skip(offset); assertEquals(offset, skipped); @@ -644,8 +643,6 @@ public class ArchiveTest extends CLIRepositoryTestCase { byte[] actual = new byte[magicBytes.length]; in.read(actual); assertArrayEquals(magicBytes, actual); - } finally { - in.close(); } } @@ -686,23 +683,19 @@ public class ArchiveTest extends CLIRepositoryTestCase { private void writeRaw(String filename, byte[] data) throws IOException { File path = new File(db.getWorkTree(), filename); - OutputStream out = new FileOutputStream(path); - try { + try (OutputStream out = new FileOutputStream(path)) { out.write(data); - } finally { - out.close(); } } private static String[] listZipEntries(byte[] zipData) throws IOException { List<String> l = new ArrayList<>(); - ZipInputStream in = new ZipInputStream( - new ByteArrayInputStream(zipData)); - - ZipEntry e; - while ((e = in.getNextEntry()) != null) - l.add(e.getName()); - in.close(); + try (ZipInputStream in = new ZipInputStream( + new ByteArrayInputStream(zipData))) { + ZipEntry e; + while ((e = in.getNextEntry()) != null) + l.add(e.getName()); + } return l.toArray(new String[l.size()]); } @@ -725,22 +718,22 @@ public class ArchiveTest extends CLIRepositoryTestCase { private String[] listTarEntries(byte[] tarData) throws Exception { List<String> l = new ArrayList<>(); Process proc = spawnAssumingCommandPresent("tar", "tf", "-"); - BufferedReader reader = readFromProcess(proc); - OutputStream out = proc.getOutputStream(); + try (BufferedReader reader = readFromProcess(proc)) { + OutputStream out = proc.getOutputStream(); - // Dump tarball to tar stdin in background - Future<?> writing = writeAsync(out, tarData); + // Dump tarball to tar stdin in background + Future<?> writing = writeAsync(out, tarData); - try { - String line; - while ((line = reader.readLine()) != null) - l.add(line); + try { + String line; + while ((line = reader.readLine()) != null) + l.add(line); - return l.toArray(new String[l.size()]); - } finally { - writing.get(); - reader.close(); - proc.destroy(); + return l.toArray(new String[l.size()]); + } finally { + writing.get(); + proc.destroy(); + } } } @@ -771,20 +764,20 @@ public class ArchiveTest extends CLIRepositoryTestCase { throws Exception { List<String> l = new ArrayList<>(); Process proc = spawnAssumingCommandPresent("tar", "Oxf", "-", path); - BufferedReader reader = readFromProcess(proc); - OutputStream out = proc.getOutputStream(); - Future<?> writing = writeAsync(out, tarData); - - try { - String line; - while ((line = reader.readLine()) != null) - l.add(line); - - return l.toArray(new String[l.size()]); - } finally { - writing.get(); - reader.close(); - proc.destroy(); + try (BufferedReader reader = readFromProcess(proc)) { + OutputStream out = proc.getOutputStream(); + Future<?> writing = writeAsync(out, tarData); + + try { + String line; + while ((line = reader.readLine()) != null) + l.add(line); + + return l.toArray(new String[l.size()]); + } finally { + writing.get(); + proc.destroy(); + } } } } |