o Updated to 'plexus-utils-3.0.23'.
authorChristian Schulte <schulte@apache.org>
Thu, 5 May 2016 03:22:53 +0000 (03:22 +0000)
committerChristian Schulte <schulte@apache.org>
Thu, 5 May 2016 03:22:53 +0000 (03:22 +0000)
o Updated to stop suppressing exceptions incorrectly when closing resources.
  Most of the time the 'IOUtils.closeQuietly' methods of 'commons-io' and
  the 'IOUtil.close' method of 'plexus-utils' are used incorrectly. They
  are meant to be used in 'finally' blocks to not suppress an exception
  already thrown in the 'try' block. The documentation of the
  'IOUtils.closeQuietly' methods explicitly contains usage examples.
  As soon as 'commons-io' or 'plexus-utils' is targetted at Java 1.7, those
  methods should get deprecated and people should be told to use the
  try-with-resources statement instead.

git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1742353 13f79535-47bb-0310-9956-ffa450edef68

pom.xml
src/main/java/org/apache/maven/plugin/ant/AntBuildWriter.java

diff --git a/pom.xml b/pom.xml
index 2cc8a19..20bb9b8 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -108,7 +108,7 @@ under the License.
     <dependency>
       <groupId>org.codehaus.plexus</groupId>
       <artifactId>plexus-utils</artifactId>
-      <version>3.0.20</version>
+      <version>3.0.23</version>
     </dependency>
     <dependency>
       <groupId>org.apache.ant</groupId>
index 9539982..4624cc3 100644 (file)
@@ -243,6 +243,8 @@ public class AntBuildWriter
         try
         {
             properties.store( os, "Generated by Maven Ant Plugin - DO NOT EDIT THIS FILE!" );
+            os.close();
+            os = null;
         }
         finally
         {
@@ -346,7 +348,7 @@ public class AntBuildWriter
 
         XmlWriterUtil.writeLineBreak( writer );
 
-        IOUtil.close( w );
+        w.close();
     }
 
     /**
@@ -415,7 +417,7 @@ public class AntBuildWriter
 
         XmlWriterUtil.writeLineBreak( writer );
 
-        IOUtil.close( w );
+        w.close();
     }
 
     /**