Refactoring/API-cleanup:
- split MethodData/FieldData into public interface and internal class
- move InternalCompilerError into the compiler source folder
- remove class ExceptionHandler and let individual plugins more directly use their own loggers.
This makes source folder jdt.core/util obsolete.
diff --git a/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTDTPlugin.java b/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTDTPlugin.java
index e6f130c..f5626b9 100644
--- a/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTDTPlugin.java
+++ b/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTDTPlugin.java
@@ -28,7 +28,6 @@
import org.eclipse.core.runtime.Plugin;
import org.eclipse.core.runtime.Status;
import org.eclipse.jdt.core.JavaCore;
-import org.eclipse.objectteams.otdt.core.exceptions.ExceptionHandler;
import org.osgi.framework.BundleContext;
@@ -147,9 +146,8 @@
return otBuildCmd;
}
- public static ExceptionHandler getExceptionHandler()
- {
- return new ExceptionHandler(PLUGIN_ID);
+ public static void logException(String message, Throwable exception) {
+ _singleton.getLog().log(createErrorStatus(message, exception));
}
public static Status createErrorStatus(String message, Throwable exception)
diff --git a/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTVariableInitializer.java b/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTVariableInitializer.java
index 0a78b51..40c2d76 100644
--- a/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTVariableInitializer.java
+++ b/plugins/org.eclipse.objectteams.otdt/src/org/eclipse/objectteams/otdt/core/ext/OTVariableInitializer.java
@@ -61,7 +61,7 @@
try {
JavaCore.setClasspathVariable(variable, new Path(installPath), new NullProgressMonitor());
} catch (JavaModelException e) {
- OTDTPlugin.getExceptionHandler().logException(e);
+ OTDTPlugin.logException("Error during initialization of classpath variable "+variable, e);
}
}
@@ -80,7 +80,7 @@
}
catch (Exception ex)
{
- OTDTPlugin.getExceptionHandler().logException(ex);
+ OTDTPlugin.logException("Error retrieving OTDT installation path", ex);
return null;
}
}
@@ -103,7 +103,7 @@
}
catch (Exception ex)
{
- OTDTPlugin.getExceptionHandler().logException(ex);
+ OTDTPlugin.logException("Error retrieving path from bundle "+bundleName, ex);
return null;
}
}