• Listen to a special audio message from Bill Roper to the Hive Workshop community (Bill is a former Vice President of Blizzard Entertainment, Producer, Designer, Musician, Voice Actor) 🔗Click here to hear his message!
  • Read Evilhog's interview with Gregory Alper, the original composer of the music for WarCraft: Orcs & Humans 🔗Click here to read the full interview.

[JASS] Memory leak - Quick Answer Plz

Status
Not open for further replies.
Level 14
Joined
Nov 25, 2004
Messages
1,185
This if ForGroupBJ function. I saw the thread where Darky28 itself recommended this construction :
JASS:
local group g = CreateGroup()
local unit u

...

set u = FirstOfGroup(g)
loop
  exitwhen u = null
  set u = FirstOfGroup(g)
  call RemoveUnit(u)     // for example
endloop

...

call DestroyGroup(g)
set g = null
set u = null
[/code]
 
Level 3
Joined
Jul 17, 2004
Messages
57
u=FirstOfGroup(g) can be inside the loop, before the exitwhen. also this wont leak (for what i know), setting u to null at the end is unneeded (it allready is) and it would make sense to have g as an auxiliar group, not using the main group.
 
Level 3
Joined
Jul 17, 2004
Messages
57
auxiliary (~help), i missed the Y, sorry.
that means when you have a group and use this it will become empty. so add the main group to the new
 
Status
Not open for further replies.
Top