Bug 264925 - [DATABINDING] cant call hashCode on Disposed Widget Observable
Summary: [DATABINDING] cant call hashCode on Disposed Widget Observable
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4.1   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: 3.5 M7   Edit
Assignee: Matthew Hall CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-14 03:45 EST by Thomas Kratz CLA
Modified: 2009-04-30 10:47 EDT (History)
1 user (show)

See Also:

Attachments
Patch (878 bytes, patch)
2009-02-16 18:53 EST, Matthew Hall CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Kratz CLA 2009-02-14 03:45:54 EST 
Build ID: HEAD

Steps To Reproduce:
1. Dispose the Observable
2. call hashCode
3.
org.eclipse.core.runtime.AssertionFailedException: assertion failed: Getter called on disposed observable org.eclipse.jface.internal.databinding.swt.SWTVetoableValueDecorator@a38b58
	at org.eclipse.core.runtime.Assert.isTrue(Assert.java:111)
	at org.eclipse.core.databinding.observable.ObservableTracker.getterCalled(ObservableTracker.java:197)
	at org.eclipse.core.databinding.observable.DecoratingObservable.getterCalled(DecoratingObservable.java:63)
	at org.eclipse.core.databinding.observable.DecoratingObservable.hashCode(DecoratingObservable.java:111)
	at java.util.HashMap.get(HashMap.java:343)
	at de.eiswind.paris.client.core.databinding.MessageManagerListener.getKey(MessageManagerListener.java:195)

More information:
I use hashCode to identify the messages in Forms MessageManager. This doesnt work anymore with the latest code. Sure one has to veto calling hashCode() ?
Comment 1 Matthew Hall CLA 2009-02-16 18:53:00 EST 
Created attachment 125835 [details]
Patch

Thomas can you please verify whether this fixes the error
Comment 2 Matthew Hall CLA 2009-02-20 11:31:21 EST 
Thomas?
Comment 3 Thomas Kratz CLA 2009-02-20 12:52:13 EST 
sorry, I stepped back to 3.4 code in the meantime. Can't verify that without greater effort.
Comment 4 Matthew Hall CLA 2009-02-20 14:33:24 EST 
Can't commit without verifying it solves the problem.  :-/
Comment 5 Matthew Hall CLA 2009-03-17 18:40:05 EDT 
Released to HEAD > 20090317.
Comment 6 Thomas Kratz CLA 2009-04-10 05:17:36 EDT 
WORKS FINE