This is an automated email from the ASF dual-hosted git repository. elharo pushed a commit to branch try in repository https://gitbox.apache.org/repos/asf/maven-apache-resources.git
commit a1156fa0bd8e47c65bd4e4f30ac5b1454dbf6d30 Author: Elliotte Rusty Harold <elh...@ibiblio.org> AuthorDate: Sun Dec 8 15:07:24 2024 -0500 Prefer try with resources --- source-release/src/test/java/org/apache/its/util/TestUtils.java | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/source-release/src/test/java/org/apache/its/util/TestUtils.java b/source-release/src/test/java/org/apache/its/util/TestUtils.java index a388913..a8bcfb6 100644 --- a/source-release/src/test/java/org/apache/its/util/TestUtils.java +++ b/source-release/src/test/java/org/apache/its/util/TestUtils.java @@ -95,20 +95,13 @@ public class TestUtils { throws ZipException, IOException { assertTrue("Assembly archive missing: " + assembly, assembly.isFile()); - ZipFile zf = null; - try { - zf = new ZipFile(assembly); - + try (ZipFile zf = new ZipFile(assembly)) { LinkedHashSet<String> pathSet = new LinkedHashSet<>(); for (Enumeration<? extends ZipEntry> enumeration = zf.entries(); enumeration.hasMoreElements(); ) { pathSet.add(enumeration.nextElement().getName()); } assertArchiveContents(required, banned, assembly.getAbsolutePath(), pathSet); - } finally { - if (zf != null) { - zf.close(); - } } }