Ticket #722 (closed: fixed)

Opened 11 years ago

Last modified 5 years ago

Fix memory leaks in Geometry

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: major Milestone: Iteration 19
Component: Keywords:
Cc: Blocked By:
Blocking: Tester:

Description

Valgrind flags up quite a significant number...

Attachments

remaining-issues.txt (18.5 KB) - added by Russell Taylor 11 years ago.
Remaining memory issues as of 31st July

Change History

comment:1 Changed 11 years ago by Russell Taylor

  • Status changed from new to assigned

comment:2 Changed 11 years ago by Russell Taylor

(In [2685]) Clear some memory leaks in Kernel. Re #722.

comment:3 Changed 11 years ago by Russell Taylor

(In [2702]) Remove some Geometry memory leaks (including ones just in tests). Re #722.

comment:4 Changed 11 years ago by Russell Taylor

(In [2755]) Few more Geometry issues and some other tidying up. Re #722.

Changed 11 years ago by Russell Taylor

Remaining memory issues as of 31st July

comment:5 Changed 11 years ago by Russell Taylor

The remaining issue is mainly around whether OpenCascade, through its internal memory manager, is holding onto a load of memory unnecessarily after its done its work. There's also something going on in ShapeFactory.

comment:6 Changed 11 years ago by Russell Taylor

(In [3027]) Address some potential memory leaks in ShapeFactory. Re #722. (Also clear a couple of gcc compiler warnings.)

comment:7 Changed 11 years ago by Russell Taylor

(In [3029]) Last plugging of a geometry leak for the time being... Re #722.

comment:8 Changed 11 years ago by Russell Taylor

  • Status changed from assigned to closed
  • Resolution set to fixed

That's all the geometry memory leaks that are highlighted by running the tests, apart from the OpenCASCADE ones (see attached document). A couple of points about these:

  • They may be spurious. OpenCASCADE has its own memory management (based on a pool-type implementation) that may not play nicely with valgrind.
  • OpenCASCADE is only used if you don't already have a cached geometry (vtp) file - i.e. rarely.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 1570

Note: See TracTickets for help on using tickets.