src/jdk.jpackage/share/classes/jdk/jpackage/internal/AbstractBundler.java
branchJDK-8200758-branch
changeset 57390 1cb722a11ead
parent 57256 d7c27451f759
child 57392 46d4b0aa4542
--- a/src/jdk.jpackage/share/classes/jdk/jpackage/internal/AbstractBundler.java	Thu Jun 06 19:07:18 2019 -0400
+++ b/src/jdk.jpackage/share/classes/jdk/jpackage/internal/AbstractBundler.java	Thu Jun 06 19:10:12 2019 -0400
@@ -67,22 +67,19 @@
             String defaultName, File result, boolean verbose, File publicRoot)
             throws IOException {
 
-        InputStream is = streamResource(publicName, category,
-                defaultName, verbose, publicRoot);
-        if (is != null) {
-            try {
+        try (InputStream is = streamResource(publicName, category,
+                defaultName, verbose, publicRoot)) {
+            if (is != null) {
                 Files.copy(is, result.toPath(),
                         StandardCopyOption.REPLACE_EXISTING);
-            } finally {
-                is.close();
-            }
-        } else {
-            if (verbose) {
-                Log.verbose(MessageFormat.format(I18N.getString(
-                        "message.no-default-resource"),
-                        defaultName == null ? "" : defaultName,
-                        category == null ? "" : "[" + category + "] ",
-                        publicName));
+            } else {
+                if (verbose) {
+                    Log.verbose(MessageFormat.format(I18N.getString(
+                            "message.no-default-resource"),
+                            defaultName == null ? "" : defaultName,
+                            category == null ? "" : "[" + category + "] ",
+                            publicName));
+                }
             }
         }
     }
@@ -91,21 +88,18 @@
             File defaultFile, File result, boolean verbose, File publicRoot)
             throws IOException {
 
-        InputStream is = streamResource(publicName, category,
-                null, verbose, publicRoot);
-        if (is != null) {
-            try {
+        try (InputStream is = streamResource(publicName, category,
+                null, verbose, publicRoot)) {
+            if (is != null) {
                 Files.copy(is, result.toPath());
-            } finally {
-                is.close();
-            }
-        } else {
-            IOUtils.copyFile(defaultFile, result);
-            if (verbose) {
-                Log.verbose(MessageFormat.format(I18N.getString(
-                        "message.using-custom-resource-from-file"),
-                        category == null ? "" : "[" + category + "] ",
-                        defaultFile.getAbsoluteFile()));
+            } else {
+                IOUtils.copyFile(defaultFile, result);
+                if (verbose) {
+                    Log.verbose(MessageFormat.format(I18N.getString(
+                            "message.using-custom-resource-from-file"),
+                            category == null ? "" : "[" + category + "] ",
+                            defaultFile.getAbsoluteFile()));
+                }
             }
         }
     }