Stomp Effects v1.0

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Use 0.03 instead of 0.01 in the loop.
Store Key(Picked unit) into an integer variable and use the variable when loading stuff from hash.
You load stuff from hash even when you might not need to.
Turn off the looping trigger when it contains no units.
Leaks Position of picked unit in the loop trigger.

 

[Magtheridon96]

0.01 seconds -> 0.03 seconds
Casting unit -> Triggering unit

 

[Bribe]

(Key(Triggering unit)) also needs to be set into a variable because you reference it a million times. The redundancy is pretty bad.

 

[Pharaoh_]

• The String part to force UnitId2String is not needed. It creates an unnecessary leak. You could use a Unit-type variable.

•

• Set deleteleak = ((Position of (Picked unit)) offset by StompDistance towards StompAngle degrees)

Leaks a location.

•

• Hashtable - Save StompCU as 7 of (Key (Picked unit)) in StompTable

Move this inside of the "Then" branch.

• Set the dummies' Combat - Death Type to "Can't raise, does not decay" and change their model to dummy.mdx, instead of using one dummy per spell. Attach a special effect on them, depending on the spell cast. http://www.wc3c.net/showthread.php?p=1007469&fromattachment=1007469#1007469

 

[BlackRangerXIII]

Nice
but it leaks, a little

 

[Ironside]

Can you make an option that each unit can only be hit by 1 missile?

 

[Adiktuz]

since the set-up part of the spells are the same, just post one of them so as not to unnecessarily populate the description...