search

heal-research / HeuristicLab

HeuristicLab - An environment for heuristic and evolutionary optimization
https://dev.heuristiclab.com
GNU General Public License v3.0
38 stars 16 forks source link

Fix memory leaks in MainForm.WindowsForms and the OperatorGraphVisualization #2006

Closed HeuristicLab-Trac-Bot closed 11 years ago

HeuristicLab-Trac-Bot commented 11 years ago

Issue migrated from trac ticket # 2006

milestone: HeuristicLab 3.3.8 | component: MainForm.WindowsForms | priority: high | resolution: done

2013-01-22 22:51:32: @mkommend created the issue

  • Menu items of the ViewHost are not disposed correctly
  • HeuristicLab.Netron contains two static class that hold references to other classes used for the OperatorGraphVisualization, which can thus not be collected.
HeuristicLab-Trac-Bot commented 11 years ago

2013-01-22 22:51:41: @mkommend changed status from new to accepted

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-22 22:53:24: @mkommend commented

r9180: Disposed menu items of the ViewHost correctly.

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-22 22:57:53: @mkommend commented

r9181: Changed GhostFactory to no longer hold a reference to the View object and completely removed the TextEditor tool as it was not in use.

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-22 23:33:07: @mkommend changed status from accepted to reviewing

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-22 23:33:07: @mkommend changed owner from @mkommend to @Shabbafru

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-24 12:45:24: @Shabbafru changed status from reviewing to assigned

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-24 12:50:21: @Shabbafru changed status from assigned to reviewing

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-24 12:50:21: @Shabbafru changed owner from @Shabbafru to @mkommend

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-24 12:50:21: @Shabbafru commented

I have reviewed/tested/profiled the changes and they work (together with the patch from #1427). I have committed the fix for the ViewContextMenuStrip as discussed in r9186.

HeuristicLab-Trac-Bot commented 11 years ago

2013-01-24 14:19:16: @mkommend changed status from reviewing to readytorelease

HeuristicLab-Trac-Bot commented 11 years ago

2013-05-10 01:42:16: @s-wagner changed status from readytorelease to closed

HeuristicLab-Trac-Bot commented 11 years ago

2013-05-10 01:42:16: @s-wagner removed resolution