You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ol...@apache.org on 2012/05/21 21:18:01 UTC
svn commit: r1341161 -
/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
Author: olamy
Date: Mon May 21 19:18:00 2012
New Revision: 1341161
URL: http://svn.apache.org/viewvc?rev=1341161&view=rev
Log:
[MPMD-144] An incomplete fix for the resource leak bugs in PmdReport.java
Modified:
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
Modified: maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java?rev=1341161&r1=1341160&r2=1341161&view=diff
==============================================================================
--- maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java (original)
+++ maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReport.java Mon May 21 19:18:00 2012
@@ -349,12 +349,12 @@ public class PmdReport
}
Writer writer = null;
-
+ FileOutputStream tStream = null;
try
{
targetDirectory.mkdirs();
File targetFile = new File( targetDirectory, "pmd." + format );
- FileOutputStream tStream = new FileOutputStream( targetFile );
+ tStream = new FileOutputStream( targetFile );
writer = new OutputStreamWriter( tStream, getOutputEncoding() );
r.setWriter( writer );
@@ -374,6 +374,7 @@ public class PmdReport
finally
{
IOUtil.close( writer );
+ IOUtil.close( tStream );
}
}