Trigger fixing

[OMGedDude]

Anyways, i made some spell called Conjure: Storm Earth and Fire. I started to make it GUI but after I finished I wanted a bit more JASS into it. But since I just learned basic JASS a few days ago I just wanted to know if there's any bugs or leaks in it . (this isn't exactly JASS btw :nuts (oh, and sorry if this is in a wrong forum, please move it if it is.)

Spoiler

Conjure Storm Earth and Fire
Events
Unit - A unit Starts the effect of an ability
Conditions
(Ability being cast) Equal to Conjure: Storm, Earth and Fire
Actions
Custom script: local unit Caster = GetTriggerUnit()
Custom script: local location CasterLoc = GetUnitLoc(Caster)
Custom script: local location SpellTarget = GetSpellTargetLoc()
Custom script: call CreateNUnitsAtLoc( 1, 'n000', GetOwningPlayer(Caster), CasterLoc, bj_UNIT_FACING )
Custom script: call UnitAddAbilityBJ( 'A001', GetLastCreatedUnit() )
Custom script: call SetUnitAbilityLevelSwapped( 'A001', GetLastCreatedUnit(), GetUnitAbilityLevelSwapped('A000', Caster) )
Custom script: call UnitApplyTimedLifeBJ( 9.00, 'BTLF', GetLastCreatedUnit() )
Custom script: call IssuePointOrderLocBJ( GetLastCreatedUnit(), "stampede", SpellTarget )
Custom script: call CreateNUnitsAtLoc( 1, 'n000', GetOwningPlayer(Caster), CasterLoc, bj_UNIT_FACING )
Custom script: call UnitAddAbilityBJ( 'A002', GetLastCreatedUnit() )
Custom script: call SetUnitAbilityLevelSwapped( 'A002', GetLastCreatedUnit(), GetUnitAbilityLevelSwapped('A000', Caster) )
Custom script: call UnitApplyTimedLifeBJ( 9.00, 'BTLF', GetLastCreatedUnit() )
Custom script: call IssuePointOrderLocBJ( GetLastCreatedUnit(), "stampede", SpellTarget )

Any leaks, bugs, suggestions? Thanks

 

[T3rm1nat0r]

nope, does not look like there are any

 

[Choppa]

Do locals leak? If so, then yours leaks.

 

[Chocobo]

Null the locals at end and remove the locations.

 

[Rheias]

3 variables that are not nulled and 2 locations that are not removed. Also, you keep using BJs when you don't need to.