URL:
<http://gna.org/patch/?884>
Summary: Memory leaks terminated
Project: Warzone Resurrection Project
Submitted by: buginator
Submitted on: Friday 12/14/2007 at 04:14
Category: Fix
Priority: 9 - Immediate
Status: None
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
_______________________________________________________
Details:
Patch comments:
Sound memory leak terminated.
Code made a bit more friendly to read/understand.
Giel (or whoever else looks at this), in the patch,
We can skip the whole routine if WZ_NOSOUND is defined
correct? It sure seems that way to me.
Even looking at cdAudio_Update(); which it calls, if that
is defined, then it skips the whole routine. Same can
be done with this IMO.
Other memleaks of note, NOT addressed by patch.
\lib\framework\printf_ext.c(76) : {14578} normal block at 0x0482EDC8, 67
bytes long.
\lib\script\event.c(487) : {472709} normal block at 0x07161F48, 255 bytes
long.
\lib\script\stack.c(126) : {1639783} normal block at 0x073BF9A0, 255 bytes
long.
Flex/bison leaks: see
http://lists.gnu.org/archive/html/help-flex/2002-06/msg00012.html
And also https://gna.org/bugs/?10197
(that covers ~16K+40bytes)
***TONS of these:
.\script_parser.y(5555) : {459498} normal block at 0x07110148, 12 bytes
long.
... way too many to list.
\src\astar.c(317) : {2403335} normal block at 0x072F9520, 24 bytes long.
\src\astar.c(317) : {2403336} normal block at 0x07396B80, 24 bytes long.
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?884>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Warzone-dev mailing list
[email protected]
https://mail.gna.org/listinfo/warzone-dev
