Apache
/
maven
mirror of https://github.com/apache/maven.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

removing JDK 5 API and adding some notes 

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@573953 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Brett Leslie Porter 2007-09-09 06:29:48 +00:00
parent e76401752d
commit 3bc1f858e1
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedderFileLogger.java
View File

@ -20,11 +20,13 @@ package org.apache.maven.embedder;
*/ */
import java.io.File; import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter; import java.io.PrintWriter;
import java.io.FileNotFoundException;
/** /**
* @author Jason van Zyl * @author Jason van Zyl
* @todo document the need to call close() once successfully constructed, otherwise file handles can be leaked. Might be good to add a finalizer too, just in case.
*/ */
public final class MavenEmbedderFileLogger public final class MavenEmbedderFileLogger
extends AbstractMavenEmbedderLogger extends AbstractMavenEmbedderLogger
@ -35,11 +37,13 @@ public final class MavenEmbedderFileLogger
{ {
try try
{ {
this.log = new PrintWriter( logFile ); this.log = new PrintWriter( new FileWriter( logFile ) );
} }
catch ( FileNotFoundException e ) catch ( IOException e )
{ {
// The client must make sure the file is valid. // The client must make sure the file is valid.
// TODO: [BP] would just throwing the IOE be better? We can't just ignore it, since that would give misleading NPE's later
throw new RuntimeException( "The embedder was unable to write to the specified log file: " + logFile, e );
} }
} }
@ -134,4 +138,4 @@ public final class MavenEmbedderFileLogger
log.close(); log.close();
} }
} }