Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 478506

Summary: NullPointerException in EventSelectionDialog.addCallEvents (257)
Product: [Modeling] Papyrus-rt Reporter: EPP Error Reports <error-reports-inbox>
Component: coreAssignee: Ansgar Radermacher <ansgar.radermacher>
Status: CLOSED MOVED QA Contact:
Severity: normal    
Priority: P3 CC: ansgar.radermacher, charles, papyrus-bugs, peter.cigehn, sredding
Version: unspecified   
Target Milestone: 0.8.0   
Hardware: All   
OS: All   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=477811
Whiteboard:

Description EPP Error Reports CLA 2015-09-28 04:33:51 EDT
The following incident was reported via the automated error reporting:


    code:                   0
    plugin:                 org.eclipse.ui_3.107.0.v20150507-1945
    message:                Unhandled event loop exception
    fingerprint:            9f9de58c
    exception class:        org.eclipse.swt.SWTException
    exception message:      Failed to execute runnable (java.lang.NullPointerException)
    number of children:     0
    
    org.eclipse.swt.SWTException: Failed to execute runnable (java.lang.NullPointerException)
    at org.eclipse.swt.SWT.error(SWT.java:4491)
    at org.eclipse.swt.SWT.error(SWT.java:4406)
    at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:138)
    at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4155)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3772)
    at org.eclipse.jface.window.Window.runEventLoop(Window.java:827)
    at org.eclipse.jface.window.Window.open(Window.java:803)
    at org.eclipse.papyrus.infra.widgets.editors.ReferenceDialog.browseAction(ReferenceDialog.java:222)
    at org.eclipse.papyrus.infra.widgets.editors.ReferenceDialog.widgetSelected(ReferenceDialog.java:445)
    at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:248)
    at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
    at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4362)
    at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1113)
    at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4180)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3769)
    at org.eclipse.jface.window.Window.runEventLoop(Window.java:827)
    at org.eclipse.jface.window.Window.open(Window.java:803)
    at org.eclipse.papyrus.views.properties.creation.PropertyEditorFactory.doEdit(PropertyEditorFactory.java:173)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.basicDoEdit(EcorePropertyEditorFactory.java:258)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory$1.call(EcorePropertyEditorFactory.java:237)
    at org.eclipse.papyrus.infra.widgets.creation.IAtomicOperationExecutor$Default$1CallableWrapper.run(IAtomicOperationExecutor.java:74)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor$1.doExecute(EObjectAdapterFactory.java:80)
    at org.eclipse.emf.transaction.RecordingCommand.execute(RecordingCommand.java:135)
    at org.eclipse.emf.workspace.EMFCommandOperation.doExecute(EMFCommandOperation.java:119)
    at org.eclipse.emf.workspace.AbstractEMFOperation.execute(AbstractEMFOperation.java:150)
    at org.eclipse.core.commands.operations.DefaultOperationHistory.execute(DefaultOperationHistory.java:516)
    at org.eclipse.papyrus.commands.CheckedOperationHistory.execute(CheckedOperationHistory.java:184)
    at org.eclipse.papyrus.commands.NotifyingWorkspaceCommandStack.doExecute(NotifyingWorkspaceCommandStack.java:261)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:165)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:219)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.execute(NestingNotifyingWorkspaceCommandStack.java:130)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.startNestedTransaction(NestingNotifyingWorkspaceCommandStack.java:85)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.execute(NestingNotifyingWorkspaceCommandStack.java:137)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor.execute(EObjectAdapterFactory.java:76)
    at org.eclipse.papyrus.infra.widgets.creation.IAtomicOperationExecutor$Default.execute(IAtomicOperationExecutor.java:87)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.doEdit(EcorePropertyEditorFactory.java:235)
    at org.eclipse.papyrus.views.properties.creation.PropertyEditorFactory.createObject(PropertyEditorFactory.java:105)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.doCreateObject(EcorePropertyEditorFactory.java:225)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.createObject(EcorePropertyEditorFactory.java:208)
    at org.eclipse.papyrus.infra.widgets.editors.MultipleValueEditor$1.run(MultipleValueEditor.java:419)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor$1.doExecute(EObjectAdapterFactory.java:80)
    at org.eclipse.emf.transaction.RecordingCommand.execute(RecordingCommand.java:135)
    at org.eclipse.emf.workspace.EMFCommandOperation.doExecute(EMFCommandOperation.java:119)
    at org.eclipse.emf.workspace.AbstractEMFOperation.execute(AbstractEMFOperation.java:150)
    at org.eclipse.core.commands.operations.DefaultOperationHistory.execute(DefaultOperationHistory.java:516)
    at org.eclipse.papyrus.commands.CheckedOperationHistory.execute(CheckedOperationHistory.java:184)
    at org.eclipse.papyrus.commands.NotifyingWorkspaceCommandStack.doExecute(NotifyingWorkspaceCommandStack.java:261)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:165)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:219)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.execute(NestingNotifyingWorkspaceCommandStack.java:130)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor.execute(EObjectAdapterFactory.java:76)
    at org.eclipse.papyrus.infra.widgets.editors.MultipleValueEditor.addAction(MultipleValueEditor.java:415)
    at org.eclipse.papyrus.infra.widgets.editors.MultipleValueEditor.widgetSelected(MultipleValueEditor.java:392)
    at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:248)
    at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
    at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4362)
    at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1113)
    at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4180)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3769)
    at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$4.run(PartRenderingEngine.java:1127)
    at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:337)
    at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1018)
    at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:156)
    at org.eclipse.ui.internal.Workbench$5.run(Workbench.java:654)
    at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:337)
    at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:598)
    at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:150)
    at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:139)
    at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
    at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
    at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
    at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:380)
    at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:235)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(null:-2)
    at sun.reflect.NativeMethodAccessorImpl.invoke(null:-1)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(null:-1)
    at java.lang.reflect.Method.invoke(null:-1)
    at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:669)
    at org.eclipse.equinox.launcher.Main.basicRun(Main.java:608)
    at org.eclipse.equinox.launcher.Main.run(Main.java:1515)
caused by: java.lang.NullPointerException: null
    at org.eclipse.papyrusrt.umlrt.tooling.ui.widgets.EventSelectionDialog.addCallEvents(EventSelectionDialog.java:257)
    at org.eclipse.papyrusrt.umlrt.tooling.ui.widgets.EventSelectionDialog$EventContentProvider.getChildren(EventSelectionDialog.java:221)
    at org.eclipse.papyrus.infra.widgets.providers.EncapsulatedContentProvider.getChildren(EncapsulatedContentProvider.java:198)
    at org.eclipse.jface.viewers.AbstractTreeViewer.getRawChildren(AbstractTreeViewer.java:1372)
    at org.eclipse.jface.viewers.TreeViewer.getRawChildren(TreeViewer.java:353)
    at org.eclipse.jface.viewers.StructuredViewer.getFilteredChildren(StructuredViewer.java:906)
    at org.eclipse.jface.viewers.AbstractTreeViewer.getSortedChildren(AbstractTreeViewer.java:617)
    at org.eclipse.jface.viewers.AbstractTreeViewer$1.run(AbstractTreeViewer.java:815)
    at org.eclipse.swt.custom.BusyIndicator.showWhile(BusyIndicator.java:70)
    at org.eclipse.jface.viewers.AbstractTreeViewer.createChildren(AbstractTreeViewer.java:791)
    at org.eclipse.jface.viewers.TreeViewer.createChildren(TreeViewer.java:611)
    at org.eclipse.jface.viewers.AbstractTreeViewer.internalExpandToLevel(AbstractTreeViewer.java:1763)
    at org.eclipse.jface.viewers.AbstractTreeViewer.internalExpandToLevel(AbstractTreeViewer.java:1773)
    at org.eclipse.jface.viewers.AbstractTreeViewer.expandToLevel(AbstractTreeViewer.java:1071)
    at org.eclipse.jface.viewers.AbstractTreeViewer.expandToLevel(AbstractTreeViewer.java:1052)
    at org.eclipse.jface.viewers.AbstractTreeViewer.expandAll(AbstractTreeViewer.java:1041)
    at org.eclipse.papyrusrt.umlrt.tooling.ui.widgets.EventSelectionDialog$EventContentProvider$1.run(EventSelectionDialog.java:182)
    at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:35)
    at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:135)
    at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4155)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3772)
    at org.eclipse.jface.window.Window.runEventLoop(Window.java:827)
    at org.eclipse.jface.window.Window.open(Window.java:803)
    at org.eclipse.papyrus.infra.widgets.editors.ReferenceDialog.browseAction(ReferenceDialog.java:222)
    at org.eclipse.papyrus.infra.widgets.editors.ReferenceDialog.widgetSelected(ReferenceDialog.java:445)
    at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:248)
    at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
    at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4362)
    at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1113)
    at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4180)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3769)
    at org.eclipse.jface.window.Window.runEventLoop(Window.java:827)
    at org.eclipse.jface.window.Window.open(Window.java:803)
    at org.eclipse.papyrus.views.properties.creation.PropertyEditorFactory.doEdit(PropertyEditorFactory.java:173)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.basicDoEdit(EcorePropertyEditorFactory.java:258)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory$1.call(EcorePropertyEditorFactory.java:237)
    at org.eclipse.papyrus.infra.widgets.creation.IAtomicOperationExecutor$Default$1CallableWrapper.run(IAtomicOperationExecutor.java:74)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor$1.doExecute(EObjectAdapterFactory.java:80)
    at org.eclipse.emf.transaction.RecordingCommand.execute(RecordingCommand.java:135)
    at org.eclipse.emf.workspace.EMFCommandOperation.doExecute(EMFCommandOperation.java:119)
    at org.eclipse.emf.workspace.AbstractEMFOperation.execute(AbstractEMFOperation.java:150)
    at org.eclipse.core.commands.operations.DefaultOperationHistory.execute(DefaultOperationHistory.java:516)
    at org.eclipse.papyrus.commands.CheckedOperationHistory.execute(CheckedOperationHistory.java:184)
    at org.eclipse.papyrus.commands.NotifyingWorkspaceCommandStack.doExecute(NotifyingWorkspaceCommandStack.java:261)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:165)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:219)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.execute(NestingNotifyingWorkspaceCommandStack.java:130)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.startNestedTransaction(NestingNotifyingWorkspaceCommandStack.java:85)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.execute(NestingNotifyingWorkspaceCommandStack.java:137)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor.execute(EObjectAdapterFactory.java:76)
    at org.eclipse.papyrus.infra.widgets.creation.IAtomicOperationExecutor$Default.execute(IAtomicOperationExecutor.java:87)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.doEdit(EcorePropertyEditorFactory.java:235)
    at org.eclipse.papyrus.views.properties.creation.PropertyEditorFactory.createObject(PropertyEditorFactory.java:105)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.doCreateObject(EcorePropertyEditorFactory.java:225)
    at org.eclipse.papyrus.views.properties.creation.EcorePropertyEditorFactory.createObject(EcorePropertyEditorFactory.java:208)
    at org.eclipse.papyrus.infra.widgets.editors.MultipleValueEditor$1.run(MultipleValueEditor.java:419)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor$1.doExecute(EObjectAdapterFactory.java:80)
    at org.eclipse.emf.transaction.RecordingCommand.execute(RecordingCommand.java:135)
    at org.eclipse.emf.workspace.EMFCommandOperation.doExecute(EMFCommandOperation.java:119)
    at org.eclipse.emf.workspace.AbstractEMFOperation.execute(AbstractEMFOperation.java:150)
    at org.eclipse.core.commands.operations.DefaultOperationHistory.execute(DefaultOperationHistory.java:516)
    at org.eclipse.papyrus.commands.CheckedOperationHistory.execute(CheckedOperationHistory.java:184)
    at org.eclipse.papyrus.commands.NotifyingWorkspaceCommandStack.doExecute(NotifyingWorkspaceCommandStack.java:261)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:165)
    at org.eclipse.emf.transaction.impl.AbstractTransactionalCommandStack.execute(AbstractTransactionalCommandStack.java:219)
    at org.eclipse.papyrus.commands.NestingNotifyingWorkspaceCommandStack.execute(NestingNotifyingWorkspaceCommandStack.java:130)
    at org.eclipse.papyrus.infra.emf.adapters.EObjectAdapterFactory$EMFAtomicOperationExecutor.execute(EObjectAdapterFactory.java:76)
    at org.eclipse.papyrus.infra.widgets.editors.MultipleValueEditor.addAction(MultipleValueEditor.java:415)
    at org.eclipse.papyrus.infra.widgets.editors.MultipleValueEditor.widgetSelected(MultipleValueEditor.java:392)
    at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:248)
    at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
    at org.eclipse.swt.widgets.Display.sendEvent(Display.java:4362)
    at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1113)
    at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4180)
    at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3769)
    at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$4.run(PartRenderingEngine.java:1127)
    at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:337)
    at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1018)
    at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:156)
    at org.eclipse.ui.internal.Workbench$5.run(Workbench.java:654)
    at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:337)
    at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:598)
    at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:150)
    at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:139)
    at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
    at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
    at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
    at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:380)
    at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:235)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(null:-2)
    at sun.reflect.NativeMethodAccessorImpl.invoke(null:-1)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(null:-1)
    at java.lang.reflect.Method.invoke(null:-1)
    at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:669)
    at org.eclipse.equinox.launcher.Main.basicRun(Main.java:608)
    at org.eclipse.equinox.launcher.Main.run(Main.java:1515)
   
  

General Information:

    reported-by:      Peter Cigéhn
    anonymous-id:     b20dc969-61bc-435c-828c-9c3681e30df3
    eclipse-build-id: 4.5.0.I20150603-2000
    eclipse-product:  org.eclipse.epp.package.modeling.product
    operating system: Windows7 6.1.0 (x86) - win32
    jre-version:      1.8.0_45-b14

The following plug-ins were present on the execution stack (*):
    1. org.eclipse.core.commands_3.7.0.v20150422-0725
    2. org.eclipse.core.databinding.observable_1.5.0.v20150422-0725
    3. org.eclipse.core.databinding_1.5.0.v20150422-0725
    4. org.eclipse.core.runtime_3.11.0.v20150405-1723
    5. org.eclipse.e4.ui.workbench_1.3.0.v20150531-1948
    6. org.eclipse.e4.ui.workbench.swt_0.13.0.v20150504-0621
    7. org.eclipse.emf.transaction_1.9.0.201506010221
    8. org.eclipse.emf_2.6.0.v20150601-0402
    9. org.eclipse.emf.workspace_1.5.1.201506010221
    10. org.eclipse.equinox.app_1.3.300.v20150423-1356
    11. org.eclipse.equinox.launcher_1.3.100.v20150511-1540
    12. org.eclipse.jface_3.11.0.v20150602-1400
    13. org.eclipse.papyrus.infra.emf_1.1.2.201509240456
    14. org.eclipse.papyrus.infra.widgets_1.1.2.201509240456
    15. org.eclipse.papyrus.views.properties_1.1.2.201509240456
    16. org.eclipse.papyrusrt.umlrt.tooling.ui_0.7.0.201509160955
    17. org.eclipse.swt_3.104.0.v20150528-0211
    18. org.eclipse.ui_3.107.0.v20150507-1945
    19. org.eclipse.ui.ide.application_1.1.0.v20150422-0725
    20. org.eclipse.ui.ide_3.11.0.v20150510-1749

Please note that:
* Messages, stacktraces, and nested status objects may be shortened.
* Bug fields like status, resolution, and whiteboard are sent
  back to reporters.
* The list of present bundles and their respective versions was
  calculated by package naming heuristics. This may or may not reflect reality.

Other Resources:
* Report: https://dev.eclipse.org/recommenders/committers/confess/#/problems/56039847e4b001019fa9855b  
* Manual: https://dev.eclipse.org/recommenders/community/confess/#/guide


Thank you for your assistance.
Your friendly error-reports-inbox.

This bug was created on behalf of ansgar.radermacher@xxxxxxxxxxxx.
Comment 1 Ansgar Radermacher CLA 2015-09-28 04:37:10 EDT
NPE can happen, if the CallEvent (of a protocol) is not associated with an interface.
Comment 2 Peter Cigehn CLA 2015-09-28 04:38:30 EDT
I have strong feeling that this might be related to the issues discussed in Bug 476237 Comment 7, which is related to that the tooling leaves dangling call events, i.e. call events which does not reference any operation (since the operation has been removed, but not call event) as reported in Bug 475901.

Even though we really should not have these inconsistencies in any protocols, the tooling maybe should handle it a bit more graceful than throwing a null pointer exception.
Comment 3 Peter Cigehn CLA 2015-09-28 04:41:21 EDT
Just an additional comment: I am not sure how much we should spend on this, since we have Bug 477811 in the pipe. Maybe the improved error handling should be done as part of that Bugzilla, instead of adding additional checks to the current dialog.
Comment 4 Simon Redding CLA 2016-09-22 13:40:36 EDT
Has this been seen again? If not, I suggest we close and re-open if it happens again as this is very old.
Comment 5 Peter Cigehn CLA 2016-09-23 04:17:00 EDT
(In reply to Simon Redding from comment #4)
> Has this been seen again? If not, I suggest we close and re-open if it
> happens again as this is very old.

I agree. Since this also seeom to happen on "corrupt" models with "corrupt" protocols with "dangling" call events, the probability of it to happen should be low now when we have the tooling in place to ensure that the protocols are handled correctly. As I stated in Comment 3, I think that any additional checks could be done as part of the work with Bug 477811.

I would also suggest that we close as resolved moved.

Ansgar, what's your view on this? Can we ensure that null pointer check is added as part of the work with Bug 477811 for this case of dangling call events, and that we simply close this one and move it over to Bug 477811?
Comment 6 Ansgar Radermacher CLA 2016-09-23 10:38:12 EDT
I agree to close, I'll verify that it is handled in the context of 477811

(In reply to Peter Cigehn from comment #5)
> (In reply to Simon Redding from comment #4)
> > Has this been seen again? If not, I suggest we close and re-open if it
> > happens again as this is very old.
> 
> I agree. Since this also seeom to happen on "corrupt" models with "corrupt"
> protocols with "dangling" call events, the probability of it to happen
> should be low now when we have the tooling in place to ensure that the
> protocols are handled correctly. As I stated in Comment 3, I think that any
> additional checks could be done as part of the work with Bug 477811.
> 
> I would also suggest that we close as resolved moved.
> 
> Ansgar, what's your view on this? Can we ensure that null pointer check is
> added as part of the work with Bug 477811 for this case of dangling call
> events, and that we simply close this one and move it over to Bug 477811?