Manaclysm v2.0

[Moderator]

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

09:47, 30th Jun 2010
Hanky:
Your spell got some leaks and isn't MUI or MPI right now. You should read the leak tutorial again since your spell is still full of leaks. Also you should read some tutorials how to make a spell MUI or at least MPI. The way you've used the hashtables didn't made the spell MUI or MPI.

 

[therash]

This is my first spell made via the trigger system

 

[WherewolfTherewolf]

Before you copy and paste the triggers to your map, be sure to change the normal melee footman unit to have no
move speed, 1000 health, to be targeted as invulnerable, and to have the name Mana Cracks. Also make an
Integer type variable named 'k' before pasting the trigger

Or better yet you could just copy and paste them into your map

-taking a look now....

K edit:

Review: (well sort of a review)
This spell is going to be rejected, I can say that already:
Reasons:
-Lot's of leaks
-Not MUI
-Doesn't support multiple levels.

Your coding is very bad atm - I'd suggest you take a look at this:
How to remove Leaks


Sorry, but I'm going to vote for rejection on this.


A note for future spell making:
Spells must be able to work with multiple units at once without interfering with each other to be acceptable - check this:
Hashtables

 

[Klingo]

Would be a plus if you could add "locust" to your dummys. Altough I recommend this
http://www.hiveworkshop.com/forums/trigger-gui-editor-tutorials-279/dynamic-values-storage-121558/ for MUI instead of Hashtable.

 

[WherewolfTherewolf]

Hashtables are more efficient and allow for MUI in more situations then Dynamic Values.

Just read that full tutorial - That's A LOT more work then using a Hash

 

[Septimus]

Hashtables are more efficient and allow for MUI in more situations then Dynamic Values.

Just read that full tutorial - That's A LOT more work then using a Hash

actually indexing was far faster than hashtable by about 400%, even though hastable are much friendly and easier to use.

 

[i_am_cuul]

actually indexing was far faster than hashtable by about 400%, even though hastable are much friendly and easier to use.

this is true; id suggest just picking one and trying it out, then when you feel extra adventurous you can make a spell with the other

 

[Kingz]

Using dynamic values gives you the maximal efficency a GUI spell can have.

Hashtables are just so messy...they use handle id's and are slow

 

[WherewolfTherewolf]

Hashtables are just so messy...they use handle id's and are slow

Hey, they added in to the editor to cover a bug people used to do essentially the same thing. And handle id's make MUI pretty simple even with GUI.

 

[Kingz]

I don't consider it fully MUI.

Try casting a spell twice in a row (a looping spell) with 1 unit because you have an item that refreshes cooldowns.

Have a nice bug right there.

 

[Darkzealot_128]

You can't use it for more than one lvl and it isn't MUI either

 

[.OmG.]

It is used much waits, and I not so like a special effect, I cannot a give more than 1/5

 

[Shmuzzy101]

effect?

wats the effect shown in the screenshot? if it is an effect can i use it for my map pls? i would give you credit of course but just wondering