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

Bug 400230

Summary: [Property View] StringEditor.dispose() is never executed and causes memory problems
Product: [Modeling] Papyrus Reporter: Calin GLITIA <calin.glitia>
Component: ViewsAssignee: Camille Letavernier <cletavernier>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: cletavernier, eclipse-bugzilla, rschnekenburger, toni.siljamaki
Version: 0.9.1   
Target Milestone: 0.10.0   
Hardware: All   
OS: All   
Whiteboard:

Description Calin GLITIA CLA 2013-02-07 11:06:15 EST 
Each time the selection changes, new org.eclipse.papyrus.views.properties.widgets.StringEditor controls are created but the controls are never disposed of when they are not needed anymore. Also, each StringEditor creates a Timer object in a new thread that keeps on executing if StringEditor.dispose() is not called.  
In the org.eclipse.papyrus.views.properties.runtime.DefaultDisplayEngine class, the disposeControls() function calls the dispose on the controls of a section but the dispose is only called on the top composites and is not passed to the children.
Comment 1 Camille Letavernier CLA 2013-02-08 08:26:52 EST 
r10095: Fixed on the trunk (0.10.X)

A dispose listener has been added on the parent composite (At the most abstract level for papyrus widgets: AbstractEditor)
Comment 2 Toni Siljamäki CLA 2013-10-09 08:02:07 EDT 
Has this one been fixed = close the bug?
Comment 3 Camille Letavernier CLA 2013-11-21 09:35:10 EST 
I close this task