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

18:43, 26th Dec 2013
Maker: The spells do not work correctly when casted more than one unit
at the same time.
You have memory leaks.

Please post the trigs and also look up the things necessary to include when uploading a spell such as documentation, import instructions etc.

1. Avoid Waits or TriggerSleepAction
2. Makes the function names decent, avoid BJs
3. Don't convert GUI to Jass and left everything as is, it's so bad because GUI is flawed in design (seen from bad structure of the code)

Here is a tutorial for helping you with jass. http://www.hiveworkshop.com/forums/...80/converting-gui-into-efficient-jass-233227/

Post the triggers into your map description, believe it or not many people don't have the WorldEditor not JNPG installed. How to easily post triggers.

Imo if you made the spells in GUI, leave them in GUI.

1. invalid preview image
2. no triggers posted
3. spell info not posted

@triggers
1. use triggering unit not casting unit
2. you leak locations
3. remove the waits
4. too long removal time on the dummy
5. first spell is not MUI since you turn the loop trigger off when casted once.
6. bad use of if blocks in the second spell
7. very bad event on the last spell.
8. store picked unit in a variable @sucking trigger

not bad for a first spell

Your spell look very nice >.<..., nice work anyway , +rep...