Need help cleaning leaks from trigger

[X-Death]

Here's my trigger for my custom spell. Just wondering if I cleaned leaks correctly?

• Crush
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Crush
  • Actions
    • Set tempPoint = (Position of (Triggering unit))
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units within 512.00 of tempPoint) and do (Actions)
      • Loop - Actions
        • Unit - Create 1 Crush Dummy Caster for (Owner of (Triggering unit)) at tempPoint facing 0.00 degrees
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Unit - Order (Last created unit) to Human Mountain King - Storm Bolt (Picked unit)
    • Custom script: call RemoveLocation(udg_tempPoint)

 

[deathismyfriend]

Leaks are cleaned fine.

This (Owner of (Triggering unit)) and (Last created unit) should be stored into a variable for efficiency.

 

[BloodDrunk]

Yes, ^^
You could also add:

• Custom script: set udg_tempPoint = null

at the end of your trigger

 

[X-Death]

Thank you guys

 

[Dr Super Good]

You could also add:

Makes no real difference. It just lets the handle ID be recycled sooner rather than later at the cost of evaluating an extra variable set. For a variable used atomically and very often like a temporary point this is worthless.

 

[BloodDrunk]

Makes no real difference. It just lets the handle ID be recycled sooner rather than later at the cost of evaluating an extra variable set. For a variable used atomically and very often like a temporary point this is worthless.

Yet, I still use it