prefuse/Prefuse

Various memory leaks

kjkrum opened this issue · 3 comments

kjkrum commented

There are many threads about memory leaks on the old SourceForge Prefuse forums. I've tracked down a few of them that were affecting my own project:

TreeLayout and its subclasses retain references to NodeItems as m_root
RadialTreeLayout retains a reference to a NodeItem as m_prevRoot
GraphDistanceFilter retains references to NodeItems in m_bfs

I've added reset() methods to clear these references. I'm sure the same thing needs to be done to many other classes. Git is an annoyance for me (I rarely use it and have to re-learn it every time I do) so until I get around to pushing my changes and creating a pull request, be aware of these potential memory leaks. I'd be perfectly happy if someone else pushed the changes and took credit for the fix.

alex-rind commented

I can take care of the "git wrangling" and I assume you can keep the credit.

Please upload the changed files and post the link.

kjkrum commented

Actually, I think I'll take this opportunity to learn how to use git via Eclipse...

kjkrum commented

This can be closed - my changes were merged.