• 🏆 Texturing Contest #33 is OPEN! Contestants must re-texture a SD unit model found in-game (Warcraft 3 Classic), recreating the unit into a peaceful NPC version. 🔗Click here to enter!
  • It's time for the first HD Modeling Contest of 2024. Join the theme discussion for Hive's HD Modeling Contest #6! Click here to post your idea!

[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