Attachment 223522 Details for Bug 394205 – Correction proposal

[patch] Correction proposal

org.eclipse.sphinx.xtendxpand_bug-394205.patch (text/plain), 3.37 KB, created by Romain Sezestre

on 2012-11-13 12:11:50 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Romain Sezestre

Created: 2012-11-13 12:11:50 EST

Size: 3.37 KB

patch

obsolete

>Index: src/org/eclipse/sphinx/xtendxpand/jobs/XtendJob.java
>===================================================================
>--- src/org/eclipse/sphinx/xtendxpand/jobs/XtendJob.java	(revision 864)
>+++ src/org/eclipse/sphinx/xtendxpand/jobs/XtendJob.java	(working copy)
>@@ -50,6 +50,7 @@
> import org.eclipse.sphinx.emf.util.EcorePlatformUtil;
> import org.eclipse.sphinx.emf.util.WorkspaceTransactionUtil;
> import org.eclipse.sphinx.platform.IExtendedPlatformConstants;
>+import org.eclipse.sphinx.platform.util.PlatformLogUtil;
> import org.eclipse.sphinx.platform.util.StatusUtil;
> import org.eclipse.sphinx.xtendxpand.XtendEvaluationRequest;
> import org.eclipse.sphinx.xtendxpand.internal.Activator;
>@@ -389,6 +390,8 @@
> 		} catch (OperationCanceledException exception) {
> 			return Status.CANCEL_STATUS;
> 		} catch (Exception ex) {
>+			// Create an entry in Error Log to inform user (https://bugs.eclipse.org/bugs/show_bug.cgi?id=394205)
>+			PlatformLogUtil.logAsError(Activator.getPlugin(), ex);
> 			return StatusUtil.createErrorStatus(Activator.getPlugin(), ex);
> 		} finally {
> 			// Always uninstall resource loader again
>Index: src/org/eclipse/sphinx/xtendxpand/jobs/CheckJob.java
>===================================================================
>--- src/org/eclipse/sphinx/xtendxpand/jobs/CheckJob.java	(revision 864)
>+++ src/org/eclipse/sphinx/xtendxpand/jobs/CheckJob.java	(working copy)
>@@ -50,6 +50,7 @@
> import org.eclipse.sphinx.emf.mwe.resources.IWorkspaceResourceLoader;
> import org.eclipse.sphinx.emf.util.EcorePlatformUtil;
> import org.eclipse.sphinx.platform.IExtendedPlatformConstants;
>+import org.eclipse.sphinx.platform.util.PlatformLogUtil;
> import org.eclipse.sphinx.platform.util.StatusUtil;
> import org.eclipse.sphinx.xtendxpand.CheckEvaluationRequest;
> import org.eclipse.sphinx.xtendxpand.XtendEvaluationRequest;
>@@ -300,6 +301,8 @@
> 		} catch (OperationCanceledException exception) {
> 			return Status.CANCEL_STATUS;
> 		} catch (Exception ex) {
>+			// Create an entry in Error Log to inform user (https://bugs.eclipse.org/bugs/show_bug.cgi?id=394205)
>+			PlatformLogUtil.logAsError(Activator.getPlugin(), ex);
> 			return StatusUtil.createErrorStatus(Activator.getPlugin(), ex);
> 		} finally {
> 			// Always uninstall resource loader again
>Index: src/org/eclipse/sphinx/xtendxpand/jobs/XpandJob.java
>===================================================================
>--- src/org/eclipse/sphinx/xtendxpand/jobs/XpandJob.java	(revision 864)
>+++ src/org/eclipse/sphinx/xtendxpand/jobs/XpandJob.java	(working copy)
>@@ -49,6 +49,7 @@
> import org.eclipse.sphinx.emf.mwe.resources.IWorkspaceResourceLoader;
> import org.eclipse.sphinx.emf.util.EcorePlatformUtil;
> import org.eclipse.sphinx.platform.IExtendedPlatformConstants;
>+import org.eclipse.sphinx.platform.util.PlatformLogUtil;
> import org.eclipse.sphinx.platform.util.StatusUtil;
> import org.eclipse.sphinx.xtendxpand.XpandEvaluationRequest;
> import org.eclipse.sphinx.xtendxpand.internal.Activator;
>@@ -388,6 +389,8 @@
> 		} catch (OperationCanceledException exception) {
> 			return Status.CANCEL_STATUS;
> 		} catch (Exception ex) {
>+			// Create an entry in Error Log to inform user (https://bugs.eclipse.org/bugs/show_bug.cgi?id=394205)
>+			PlatformLogUtil.logAsError(Activator.getPlugin(), ex);
> 			return StatusUtil.createErrorStatus(Activator.getPlugin(), ex);
> 		} finally {
> 			// Always uninstall resource loader again

Flags:

idydieng: iplog+

Actions: View | Diff

Attachments on bug 394205: 223522