Moderator
M
Moderator
12th Dec 2015
IcemanBo: For long time as NeedsFix. Rejected.
Bribe:
Recommend caching some variables before entering loops - (Owner of Caster) should be set to a variable (Could literally be simplified as (Triggering player))
Same with ((Current flying height of Target) + 70.00) - obviously this isn't going to change from within the loop, so qualify it before entering the loop so that you don't waste function calls & math.
In Death Penalty Spin, you should move "Custom script: call RemoveLocation(udg_Point3)" into the if-block where Point3 was set, in order to avoid "removing" a location which was already removed (null). This is a logical improvement and a slight efficiency improvement.
You are using custom value of units in this trigger as well, which conflicts with other systems that might use it. Please do everything in the hashtable (good) or get a GUI unit indexing system (great).
Maker:
Leaks here
IcemanBo: For long time as NeedsFix. Rejected.
Bribe:
Recommend caching some variables before entering loops - (Owner of Caster) should be set to a variable (Could literally be simplified as (Triggering player))
Same with ((Current flying height of Target) + 70.00) - obviously this isn't going to change from within the loop, so qualify it before entering the loop so that you don't waste function calls & math.
In Death Penalty Spin, you should move "Custom script: call RemoveLocation(udg_Point3)" into the if-block where Point3 was set, in order to avoid "removing" a location which was already removed (null). This is a logical improvement and a slight efficiency improvement.
You are using custom value of units in this trigger as well, which conflicts with other systems that might use it. Please do everything in the hashtable (good) or get a GUI unit indexing system (great).
Maker:
Leaks here
-
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
-
If - Conditions
- LoopsForASFX2 Equal to PlaySFXInteger
-
Then - Actions
- Set Point3 = (Point1 offset by Distance towards ((Angle from Point1 to Point2) + (Real((Custom value of (Picked unit))))) degrees)
- -------- NOTE 2: Change your SFX played HERE! --------
- Special Effect - Create a special effect at Point3 using Abilities\Spells\Undead\OrbOfDeath\AnnihilationMissile.mdl
- Special Effect - Destroy (Last created special effect)
- Else - Actions
-
If - Conditions