| Summary: | [Forms] AbstractHyperlink.handleActivate(...) improved disposed check | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Chris Aniszczyk <caniszczyk> |
| Component: | User Assistance | Assignee: | platform-ua-inbox <platform-ua-inbox> |
| Status: | RESOLVED DUPLICATE | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | agarcher |
| Version: | 3.5 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Mac OS X - Carbon (unsup.) | ||
| Whiteboard: | |||
*** This bug has been marked as a duplicate of bug 250505 *** |
I believe the triggerAccessible() method should be encased in the isDisposed checked. if (!isDisposed()) setCursor(FormsResources.getHandCursor()); triggerAccessible(); Otherwise, you can get an exception: at org.eclipse.swt.SWT.error(SWT.java:3777) at org.eclipse.swt.SWT.error(SWT.java:3695) at org.eclipse.swt.SWT.error(SWT.java:3666) at org.eclipse.swt.widgets.Widget.error(Widget.java:446) at org.eclipse.swt.widgets.Widget.checkWidget(Widget.java:384) at org.eclipse.swt.widgets.Control.getAccessible(Control.java:602) at org.eclipse.ui.forms.widgets.AbstractHyperlink?.triggerAccessible(AbstractHyperlink?.java:235) at org.eclipse.ui.forms.widgets.AbstractHyperlink?.handleActivate(AbstractHyperlink?.java:231) at org.eclipse.ui.forms.widgets.AbstractHyperlink?.handleMouseUp(AbstractHyperlink?.java:320) at org.eclipse.ui.forms.widgets.AbstractHyperlink?.access$2(AbstractHyperlink?.java:304) at org.eclipse.ui.forms.widgets.AbstractHyperlink?$4.handleEvent(AbstractHyperlink?.java:119)