Re: [Plib-devel] Memory leak in AC loader

[Bram S. <br...@sa...>]

Durk Talsma wrote:
> Dear plib developers,
> 
> Recently, I've been trying to hunt down memory leaks in FlightGear, and ran it 
> through valgrind. One of the results coming out of valgrind is that there 
> appears to be a memory leak in plib's AC3d Model loader. 
> 
> In ssgLoadAC.cxx, there is a static variable mlist, which is a static array of 
> pointers to _ssgMaterial objects:

good catch.

I am assuming that there is still no SVN repo, so I committed your fix
to the CVS tree.

I don't know how fast it will appear on user cvs though... it used to be that
developer changes to cvs would take quite some time to appear on user cvs.

   Bram

PS: It would be best ofcourse, to get rid of the MAX_MATERIALS (1000)
altogether. This fixed size mlist is really ugly.