You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jv...@apache.org on 2008/07/17 23:06:18 UTC

svn commit: r677726 - /maven/plugins/trunk/maven-shade-plugin/src/main/java/org/apache/maven/plugins/shade/resource/ManifestResourceTransformer.java

Author: jvanzyl
Date: Thu Jul 17 14:06:18 2008
New Revision: 677726

URL: http://svn.apache.org/viewvc?rev=677726&view=rev
Log:
o close the manifest stream we snagged. use the IOUtil for it all

Modified:
    maven/plugins/trunk/maven-shade-plugin/src/main/java/org/apache/maven/plugins/shade/resource/ManifestResourceTransformer.java

Modified: maven/plugins/trunk/maven-shade-plugin/src/main/java/org/apache/maven/plugins/shade/resource/ManifestResourceTransformer.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-shade-plugin/src/main/java/org/apache/maven/plugins/shade/resource/ManifestResourceTransformer.java?rev=677726&r1=677725&r2=677726&view=diff
==============================================================================
--- maven/plugins/trunk/maven-shade-plugin/src/main/java/org/apache/maven/plugins/shade/resource/ManifestResourceTransformer.java (original)
+++ maven/plugins/trunk/maven-shade-plugin/src/main/java/org/apache/maven/plugins/shade/resource/ManifestResourceTransformer.java Thu Jul 17 14:06:18 2008
@@ -67,6 +67,7 @@
         {
             manifest = new Manifest( is );
             manifestDiscovered = true;
+            IOUtil.close(  is );
         }
     }
 
@@ -109,6 +110,8 @@
         }       
         InputStream is = new StringInputStream( sb.toString() );
         IOUtil.copy( is, jos );
-        is.close();
+        
+        // Close up all the streams.
+        IOUtil.close( is );
     }    
 }