From 7627a966ee4283bebb43ff4f0b6eeb84236a6019 Mon Sep 17 00:00:00 2001
From: Jacobus Martinus Kruithof <jkf@apache.org>
Date: Sat, 30 Jun 2007 21:59:12 +0000
Subject: [PATCH] Reviewed Jar (and parts of Zip) because of BR 42632

git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@552218 13f79535-47bb-0310-9956-ffa450edef68
---
 src/main/org/apache/tools/ant/taskdefs/Jar.java | 10 +++-------
 src/main/org/apache/tools/ant/taskdefs/Zip.java |  5 ++++-
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/main/org/apache/tools/ant/taskdefs/Jar.java b/src/main/org/apache/tools/ant/taskdefs/Jar.java
index df1e593ab..f10098674 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Jar.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Jar.java
@@ -150,10 +150,6 @@ public class Jar extends Zip {
         JarMarker.getInstance()
     };
 
-    // CheckStyle:VisibilityModifier OFF - bc
-    protected String emptyBehavior = "create";
-    // CheckStyle:VisibilityModifier ON
-
     /** constructor */
     public Jar() {
         super();
@@ -456,7 +452,7 @@ public class Jar extends Zip {
         throws IOException {
         for (Enumeration e = manifest.getWarnings();
              e.hasMoreElements();) {
-            log("Manifest warning: " + (String) e.nextElement(),
+            log("Manifest warning: " + e.nextElement(),
                 Project.MSG_WARN);
         }
 
@@ -467,7 +463,7 @@ public class Jar extends Zip {
         OutputStreamWriter osw = new OutputStreamWriter(baos, Manifest.JAR_ENCODING);
         PrintWriter writer = new PrintWriter(osw);
         manifest.write(writer);
-        writer.flush();
+        writer.close();
 
         ByteArrayInputStream bais =
             new ByteArrayInputStream(baos.toByteArray());
@@ -549,7 +545,7 @@ public class Jar extends Zip {
             }
         }
 
-        writer.flush();
+        writer.close();
         ByteArrayInputStream bais =
             new ByteArrayInputStream(baos.toByteArray());
         super.zipFile(bais, zOut, INDEX_NAME, System.currentTimeMillis(), null,
diff --git a/src/main/org/apache/tools/ant/taskdefs/Zip.java b/src/main/org/apache/tools/ant/taskdefs/Zip.java
index 6e3e574ff..79d5e642d 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Zip.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Zip.java
@@ -759,13 +759,16 @@ public class Zip extends MatchingTask {
                         if (keepCompression) {
                             doCompress = (ze.getMethod() == ZipEntry.DEFLATED);
                         }
+                        InputStream is = null;
                         try {
-                            zipFile(zf.getInputStream(ze), zOut, prefix + name,
+                            is = zf.getInputStream(ze);
+                            zipFile(is, zOut, prefix + name,
                                     ze.getTime(), zfs.getSrc(getProject()),
                                     zfs.hasFileModeBeenSet() ? fileMode
                                     : ze.getUnixMode());
                         } finally {
                             doCompress = oldCompress;
+                            FileUtils.close(is);
                         }
                     }
                     } else {