You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@clerezza.apache.org by re...@apache.org on 2010/01/21 14:44:05 UTC

svn commit: r901707 - /incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.maven-ontologies-plugin/src/main/java/org/apache/clerezza/ontologiesplugin/OntologyJavaClassCreatorMojo.java

Author: reto
Date: Thu Jan 21 13:44:04 2010
New Revision: 901707

URL: http://svn.apache.org/viewvc?rev=901707&view=rev
Log:
CLEREZZA-84: more fine grained exception handling, finally block only applied when out id not null

Modified:
    incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.maven-ontologies-plugin/src/main/java/org/apache/clerezza/ontologiesplugin/OntologyJavaClassCreatorMojo.java

Modified: incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.maven-ontologies-plugin/src/main/java/org/apache/clerezza/ontologiesplugin/OntologyJavaClassCreatorMojo.java
URL: http://svn.apache.org/viewvc/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.maven-ontologies-plugin/src/main/java/org/apache/clerezza/ontologiesplugin/OntologyJavaClassCreatorMojo.java?rev=901707&r1=901706&r2=901707&view=diff
==============================================================================
--- incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.maven-ontologies-plugin/src/main/java/org/apache/clerezza/ontologiesplugin/OntologyJavaClassCreatorMojo.java (original)
+++ incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.maven-ontologies-plugin/src/main/java/org/apache/clerezza/ontologiesplugin/OntologyJavaClassCreatorMojo.java Thu Jan 21 13:44:04 2010
@@ -19,6 +19,7 @@
 package org.apache.clerezza.ontologiesplugin;
 
 import java.io.File;
+import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.net.MalformedURLException;
@@ -27,6 +28,8 @@
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Properties;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -165,22 +168,30 @@
 			}
 		};
 
-		PrintWriter out = null;
 		SchemaGen schemaGen;
 		try {
 			schemaGen = new SchemaGen(arguments);
-			String rootPath = baseDir + File.separator + "target"
-					+ File.separator + "generated-sources" + File.separator
-					+ "main" + File.separator + "java" + File.separator;
-			File dir = new File(rootPath + pathToJavaClass);
-			dir.mkdirs();
-			out = new PrintWriter(new File(rootPath + pathToJavaClass
-					+ className + ".java"));
-			schemaGen.writeClass(out);
 		} catch (IOException e) {
 			getLog().error(e.getMessage(), e);
+			return;
 		} catch (URISyntaxException e) {
 			getLog().error(e.getMessage(), e);
+			return;
+		}
+		String rootPath = baseDir + File.separator + "target"
+				+ File.separator + "generated-sources" + File.separator
+				+ "main" + File.separator + "java" + File.separator;
+		File dir = new File(rootPath + pathToJavaClass);
+		dir.mkdirs();
+		PrintWriter out = null;
+		try {
+			out = new PrintWriter(new File(rootPath + pathToJavaClass + className + ".java"));
+		} catch (FileNotFoundException e) {
+			getLog().error(e.getMessage(), e);
+			return;
+		}
+		try {
+			schemaGen.writeClass(out);
 		} finally {
 			out.flush();
 		}