Skip to main content

Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Eclipse Projects » Standard Widget Toolkit (SWT) » New Data binding: Missing event deregistration?
New Data binding: Missing event deregistration? [message #465784] Tue, 20 December 2005 13:37 Go to next message
Daniel Krügler is currently offline Daniel KrüglerFriend
Messages: 853
Registered: July 2009
Senior Member

I am just studying the new and very promising data binding
library. In org.eclipse.jface.databinding.internal.NestedUpdatableValue
I stumbled across the c'tor:

public NestedUpdatableValue(IDataBindingContext databindingContext,
final IUpdatableValue outerUpdatableValue,
Object feature, Class featureType) {
this.databindingContext = databindingContext;
this.feature = feature;
this.featureType = featureType;
IChangeListener outerChangeListener = new IChangeListener() {
public void handleChange(ChangeEvent changeEvent) {
Object oldValue = getValue();

fireChangeEvent(ChangeEvent.CHANGE, oldValue, getValue());

which registers an IChangeListener on its argument outerUpdatableValue.
According to usual resource management rules, the class
NestedUpdatableValue should ensure that removeChangeListener is
correctly envoked, but I could not find any corresponding
deregistration. Is this a bug or just my ignorance?

Greetings from Bremen,

Daniel Krügler
Re: New Data binding: Missing event deregistration? [message #465788 is a reply to message #465784] Tue, 20 December 2005 14:11 Go to previous message
Paul Webster is currently offline Paul WebsterFriend
Messages: 6859
Registered: July 2009
Location: Ottawa
Senior Member

This is probably more appropriate for eclipse.platform.


Previous Topic:Don't see colors running SWT application under Linux SUSE
Next Topic:TableWrapLayout bug
Goto Forum:

Current Time: Thu Sep 19 09:04:26 GMT 2019

Powered by FUDForum. Page generated in 0.01863 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top