Eclipse Community Forums: Graphiti » Unregistering images when closing the diagram editor

Help

Home

Home » Modeling » Graphiti » Unregistering images when closing the diagram editor

Show: Today's Messages :: Show Polls :: Message Navigator

Unregistering images when closing the diagram editor [message #648621]

Thu, 13 January 2011 17:18

Shenxue Zhou

Messages: 60
Registered: July 2009

Member

There seems no way to un-register images when closing diagram editor. DiagramEditorInternal's dispose() method would call ConfigurationProvider's dispose() method which in turn calls DiagramTypeProvider's dispose() method. But I've not found any direct/indirect references to the image provider during diagram editor's dispose call.

In fact, I experimented with setting a break point on the constructor of my image provider. If I closed my diagram editor and re-opened it, I didn't get a break on the image provider's constructor. So the image provider is only initialized once and being held on to even after the diagram editor is disposed. This seems to be memory leaks.

Any idea on what's going on here?

Thanks!

Shenxue

Report message to a moderator

Re: Unregistering images when closing the diagram editor [message #648721 is a reply to message #648621]

Fri, 14 January 2011 08:50

Michael Wenz

Messages: 1931
Registered: July 2009
Location: Walldorf, Germany

Senior Member

For performance and memory consumption reasons we decided to hoold the image
provider at the diagram type and not at the editor instance. This has the
advantage that only for the first start of an editor for a certain diagram
type the images need to be loaded and all subsequent starts are faster.
Besides memory is only allocated once and not for each editor instance.

The downside is of course that the images stay in memory after closing the
last editor, because the framework cannot tell if the user will open another
diagram editor of this kind in the future or not. If you see this as a bug,
please open a bugzilla for that.

Michael

"Shenxue Zhou" <shenxue.zhou@oracle.com> wrote in message
news:ignbqq$b8k$1@news.eclipse.org...
> There seems no way to un-register images when closing diagram editor.
> DiagramEditorInternal's dispose() method would call
> ConfigurationProvider's dispose() method which in turn calls
> DiagramTypeProvider's dispose() method. But I've not found any
> direct/indirect references to the image provider during diagram editor's
> dispose call.
> In fact, I experimented with setting a break point on the constructor of
> my image provider. If I closed my diagram editor and re-opened it, I
> didn't get a break on the image provider's constructor. So the image
> provider is only initialized once and being held on to even after the
> diagram editor is disposed. This seems to be memory leaks.
>
> Any idea on what's going on here?
>
> Thanks!
>
> Shenxue

Report message to a moderator

Re: Unregistering images when closing the diagram editor [message #648848 is a reply to message #648621]

Fri, 14 January 2011 17:39

Shenxue Zhou

Messages: 60
Registered: July 2009

Member

I've opened https://bugs.eclipse.org/bugs/show_bug.cgi?id=334402 to track this issue. I believe it's a good programming practice to release all the memories upon closing the editor.

Report message to a moderator

Previous Topic:	Graphiti releases download site url format
Next Topic:	Create object on dropping connection to diagram

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Thu Apr 18 04:03:16 GMT 2024

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter