Question About Leaks

[Landscaper]

I read a tutorial about basic memory leaks by Ralle (https://www.hiveworkshop.com/forums/trigger-gui-editor-tutorials-279/basic-memory-leaks-5889/) and I've got one question:
Do I need to remove the location, unit group, etc. if I'm using the trigger multiple times?

 

[Dr. Boom]

Moin moin =)

That's a very good questions, because there are a difference... I don't know why but I notice this some time ago.
If you got for example the trigger:

• Test
  • Events
    • Time - Every 0.03 seconds of game time
  • Conditions
  • Actions
    • Set TempLoc = (Center of (Playable map area))
    • Unit - Create 1 Footman for Player 1 (Red) at TempLoc facing Default building facing (270.0) degrees
    • Custom script: call RemoveLocation(udg_TempLoc)

then you have to remove the leak, else you got every 0.03 seconds a leak.

But, I can't give an example now, I notice it, that if I use this example above in a MUI trigger ( GUI in the loop trigger ) then the trigger doesn't work anymore correctly. ( Maybe other conditions ... )

So I don't know it exactly but actually you have to clear the leak in trigger you multiple use.

Greetings
~ The Bomb King > Dr. Boom

 

[Pharaoh_]

Of course, once the leak is created, it needs to be destroyed, else, a bit of memory will be on hold. Visit this page as well: https://www.hiveworkshop.com/forums/triggers-scripts-269/things-leak-35124/

 

[watermelon_1234]

If you set a location variable to one point only, you only need to remove it once regardless of how many times you refer to that variable in your trigger. However, if you use it for more than one location, you'll need to remove it more than once.
Ex:
This is okay:

• Set Temp_Loc = (Position of (Triggering unit))
• Unit - Create 1 Footman for (Owner of (Triggering unit)) at Temp_Loc facing Default building facing degrees
• Unit - Create 1 Archer for (Owner of (Triggering unit)) at Temp_Loc facing Default building facing degrees
• Custom script: call RemoveLocation(udg_Temp_Loc)

Not this:

• Set Temp_Loc = (Position of (Triggering unit))
• Unit - Create 1 Footman for (Owner of (Triggering unit)) at Temp_Loc facing Default building facing degrees
• Set Temp_Loc = (Position of (Target unit of ability being cast))
• Unit - Create 1 Footman for (Owner of (Triggering unit)) at Temp_Loc facing Default building facing degrees
• Custom script: call RemoveLocation(udg_Temp_Loc)

Do this:

• Set Temp_Loc = (Position of (Triggering unit))
• Unit - Create 1 Footman for (Owner of (Triggering unit)) at Temp_Loc facing Default building facing degrees
• Custom script: call RemoveLocation(udg_Temp_Loc)
• Set Temp_Loc = (Position of (Target unit of ability being cast))
• Unit - Create 1 Footman for (Owner of (Triggering unit)) at Temp_Loc facing Default building facing degrees
• Custom script: call RemoveLocation(udg_Temp_Loc)

 

[Landscaper]

Thanks guys, you really helped. +rep to each and every.