[Trigger] Does this leak?

[PurpleHephalump]

• Unit - A unit enters Region 018 <gen>
• Set Point = (Center of Region 006 <gen>)
• Unit - Kill (Triggering unit)
• Unit - Create 1 Hunter for (Owner of (Entering unit)) at (Center of Region 006 <gen>) facing Default building facing degrees
• Camera - Pan camera for (Owner of (Triggering unit)) to (Position of (Last created unit)) over 1.00 seconds
• Hero - Create Novice Blade and give it to (Last created unit)
• Set SaveLoadPlayerHeroSingle[(Player number of (Owner of (Triggering unit)))] = (Last created unit)
• Wait 1.00 seconds
• Unit - Create 1 Backpack for (Owner of (Entering unit)) at (Center of Region 006 <gen>) facing Default building facing degrees
• Set SaveLoadHasLoaded[(Player number of (Owner of (Triggering unit)))] = True
• Custom script: call RemoveLocation(udg_Point)

im just wondering if this leaks does anyone know?

 

[PurgeandFire]

You have to use "Point" instead of Center of Region 006.

Since you are setting Point to that variable, you have to use it to avoid leaking it.

 

[Valdez]

It doesn't even look like he uses the variable at all so thus just deleting the variable from the trigger would be a better idea IMO..

 

[Gespenst]

Ah in the "kill unit section" the a unit corpse will spawn, unless you set the unit not to decay, but if you wanted that to happen no problem.

If you didnt, use "replace" or "remove" unit strings.

 

[redmarine]

leaks...

• Unit - Create 1 Hunter for (Owner of (Entering unit)) at (Center of Region 006 <gen>) facing Default building facing degrees

Corrected:

• Unit - Create 1 Hunter for (Owner of (Entering unit)) at (Point) facing Default building facing degrees

 

[Marcelo Hossomi]

You could also use temporary variables for (Triggering Unit), (Owner of (Triggering Unit)), (Last Created Unit) to avoid repeating it so many times. You can use the same variables all the time, EXCEPT after waits...

 

[redmarine]

If he uses waits then he should use locals? I believe it is Jass only.