[WyrmTools-Developers] [ wyrmtools-Bugs-1757881 ] Layout fragments are leaked

[SourceForge.net <no...@so...>]

Bugs item #1757881, was opened at 2007-07-20 23:12
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=724978&aid=1757881&group_id=132696

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: User Interface
Group: Pre-Alpha
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Wyrm Tools (wyrmtools)
Assigned to: Wyrm Tools (wyrmtools)
Summary: Layout fragments are leaked

Initial Comment:
The various implementations of org.wyrmtools.ui.model.ILayoutFragment implement IModelListener.  Although their dispose() implementation unregisters themselves as model listeners, this method is never called.  (This is because all structure providers assume their elements are IElements, which should not have their dispose() method invoked arbirtrarily.)  Thus, every layout fragment ever created is leaked.

This not only causes memory usage to increase, but the model listener list becomes so huge that UI updates take increasingly longer the longer the application is running.

Weak references should be used for layout fragments.  This could be implemented by passing a proxy that wraps the layout fragment in a WeakReference to the IModel.addModelListener() method.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=724978&aid=1757881&group_id=132696