Opened 5 years ago

Closed 4 years ago

#2006 closed defect (done)

Fix memory leaks in MainForm.WindowsForms and the OperatorGraphVisualization

Reported by: mkommend Owned by: mkommend
Priority: high Milestone: HeuristicLab 3.3.8
Component: MainForm.WindowsForms Version: 3.3.8
Keywords: Cc:

Description

  • 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.

Change History (8)

comment:1 Changed 5 years ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 5 years ago by mkommend

r9180: Disposed menu items of the ViewHost correctly.

comment:3 Changed 5 years ago by mkommend

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.

comment:4 Changed 5 years ago by mkommend

  • Owner changed from mkommend to ascheibe
  • Status changed from accepted to reviewing

comment:5 Changed 5 years ago by ascheibe

  • Status changed from reviewing to assigned

comment:6 Changed 5 years ago by ascheibe

  • Owner changed from ascheibe to mkommend
  • Status changed from assigned to reviewing

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.

comment:7 Changed 5 years ago by mkommend

  • Status changed from reviewing to readytorelease

comment:8 Changed 4 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.7 to 3.3.8
Note: See TracTickets for help on using tickets.