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 );
}
}