• 🏆 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!
  • 🏆 Hive's 6th HD Modeling Contest: Mechanical is now open! Design and model a mechanical creature, mechanized animal, a futuristic robotic being, or anything else your imagination can tinker with! 📅 Submissions close on June 30, 2024. Don't miss this opportunity to let your creativity shine! Enter now and show us your mechanical masterpiece! 🔗 Click here to enter!

[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