You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by jo...@apache.org on 2011/10/10 15:39:43 UTC

svn commit: r1180953 - in /incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor: ModelUtil.java OpenNLPPlugin.java

Author: joern
Date: Mon Oct 10 13:39:43 2011
New Revision: 1180953

URL: http://svn.apache.org/viewvc?rev=1180953&view=rev
Log:
OPENNLP-315 Added log methods.

Modified:
    incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/ModelUtil.java
    incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/OpenNLPPlugin.java

Modified: incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/ModelUtil.java
URL: http://svn.apache.org/viewvc/incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/ModelUtil.java?rev=1180953&r1=1180952&r2=1180953&view=diff
==============================================================================
--- incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/ModelUtil.java (original)
+++ incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/ModelUtil.java Mon Oct 10 13:39:43 2011
@@ -41,8 +41,8 @@ public class ModelUtil {
       try {
         modelIn = modelFile.getContents();
       } catch (CoreException e) {
-        // TODO: Handle this exception correctly ...
-        e.printStackTrace();
+        // TODO: Improve error handling here ...
+        OpenNLPPlugin.log(e);      
       }
     }
     

Modified: incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/OpenNLPPlugin.java
URL: http://svn.apache.org/viewvc/incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/OpenNLPPlugin.java?rev=1180953&r1=1180952&r2=1180953&view=diff
==============================================================================
--- incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/OpenNLPPlugin.java (original)
+++ incubator/opennlp/sandbox/caseditor-opennlp-plugin/src/main/java/org/apache/opennlp/caseditor/OpenNLPPlugin.java Mon Oct 10 13:39:43 2011
@@ -17,6 +17,8 @@
 
 package org.apache.opennlp.caseditor;
 
+import org.eclipse.core.runtime.IStatus;
+import org.eclipse.core.runtime.Status;
 import org.eclipse.ui.plugin.AbstractUIPlugin;
 import org.osgi.framework.BundleContext;
 
@@ -41,5 +43,18 @@ public class OpenNLPPlugin extends Abstr
 	public static OpenNLPPlugin getDefault() {
 		return instance;
 	}
-	
+
+  public static void log(Throwable t) {
+    getDefault().getLog().log(
+        new Status(IStatus.ERROR, ID, IStatus.OK, t.getMessage(), t));
+  }
+
+  public static void log(String message, Throwable t) {
+    getDefault().getLog().log(
+        new Status(IStatus.ERROR, ID, IStatus.OK, message, t));
+  }
+
+  public static void logError(String message) {
+    getDefault().getLog().log(new Status(IStatus.ERROR, ID, message));
+  }
 }