[JASS] Memory leak - Quick Answer Plz

Status
Not open for further replies.
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]
 
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.
 
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.
Back
Top