Ticket #7819 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Memory leaks reported by visual leak detector that need fixing

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

Various leaks in algorithm intialization, particularly those using the IPropertySettings objects.

Others include the dock and fit property browser and undoubtedly others.

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

Fix memory leak when overwriting a property settings object.

The previous one must be deleted first. Refs #7819

Changeset: 0325334fa636edefb36a5cffe6b81d1669c450a2

comment:2 Changed 7 years ago by Martyn Gigg

Fix small mem leak in Stitch1D with validator not being deleted.

Refs #7819

Changeset: 0a9b9170d371e4aba22ae740b1b382a6a4f9f386

comment:3 Changed 7 years ago by Martyn Gigg

Fix various small leaks in UI layer.

Refs #7819

Changeset: a4d2ec5082eec48c7d057ce6e0f6efb708d1799c

comment:4 Changed 7 years ago by Martyn Gigg

Initialize CompositeFunction base class properly.

Refs #7819

Changeset: 7c33cfad781d0f7ab62a387da11fcf8f1c43d200

comment:5 Changed 7 years ago by Martyn Gigg

Remove custom defines of declspec in API & TableWorkspace

Not strictly related to this ticket but there was no reason for this to be duplicated. Refs #7819

Changeset: 3f6f6cdc4eed901021988556ec31bb7a61ad8d37

comment:6 Changed 7 years ago by Martyn Gigg

Fix hang when MantidPlot shuts down on RHEL6

Refs #7819

Changeset: b58cd57e07e0cef93c4dd51be0eeaab0bce1306a

comment:7 Changed 7 years ago by Martyn Gigg

  • Status changed from inprogress to verify
  • Resolution set to fixed

Branch: bugfix/7819_fix_memory_leaks

Tester: The aim was to clear up some obvious leaks that were polluting the output. They are not big things but kept cropping up in the report and had obvious fixes. Tests will have to be that the tests passe and by inspection.

comment:8 Changed 7 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:9 Changed 7 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/7819_fix_memory_leaks'

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8664

Note: See TracTickets for help on using tickets.