really easy question about leaks

[HellslayerX15]

if i set a temporary point several times in a trigger without waits do i need to remove them each time i create them or just in the end of the trigger?

 

[Drenith]

i'm 99% sure its every time, besides its not like being extra careful can hurt

 

[Marcelo Hossomi]

Every time after it's used.

 

[HellslayerX15]

i'm 99% sure its every time, besides its not like being extra careful can hurt

yes it hurts it takes extra space and is boring ;[ sorry for not giving rep but i felt like the other guy had a more clear answer ^^ thanks though

 

[HellslayerX15]

i just had a little thought, let's say i have a loop wich each time checks distance between temp_point and temp_point_1 or something, do i then each loop have to remove the leak? even if i dont set the points to something else?

 

[Coolty44]

Well if your comparing a point to another point, then you might want to save the one that your trying to get closer to until its no longer refrenced to, but if you change the other point, then remove it as the last thing in the loop, and set it at the top, since Blizzard never really thought about leaks and such in GUI

 

[HellslayerX15]

so the answer is yes? that would screw things up a bit for me :O

 

[ACDS2]

I think a point should be removed after each "set" rather than after each use..

 

[Marcelo Hossomi]

Huh, I read again my post and maybe it could cause some confusion...
I meant it should be removed after you don't need it anymore.

A point is like an object in the map that must be removed when it's useless. And, whenever you use "Set var = point", you're creating a new "object". So, if you don't remove it before setting the variable again, you won't be able to do so anymore.

Therefore, you have to remove/set point variables in a loop if you need to change it. If the point remains constant during the whole loop, set it before looping and remove after doing so. Like this:

• -------- Creating 6 units in random points --------
• For each integer (Integer A) from 1 to 6 do (Actions)
  • Loop - Actions
    • Set ThePoint = (Random point in (region))
    • Unit - Create 1 Footman at ThePoint [...]
    • Custom Script: call RemoveLocation( udg_ThePoint )

• -------- Creating 6 units at the same point --------
• Set ThePoint = (Random point in (region))
• For each integer (Integer A) from 1 to 6 do (Actions)
  • Loop - Actions
    • Unit - Create 1 Footman at ThePoint [...]
• Custom Script: call RemoveLocation( udg_ThePoint )

 

[HellslayerX15]

well that's what i thought but i did it another way in some triggers now, need to remove some shit now