[GUI] Electrify v1e

[Moderator]

19:33, 16th Jul 2012
Magtheridon96:

Status: Approved.

The configuration is exceptional.
The spell is also quite original.
The indexing method is fine.

You need to merge the periodic triggers. Just take the code
of the second one and put it below the code of the first one.

edit
Another thing I would recommend is caching the AoE into a variable
so you don't have to recalculate it each time you do one iteration of
the spell.

edit
Excellent work.

 

[Tank-Commander]

Those loops triggers might as well be one trigger as opposed to two, also what's the purpose of the dummy which is created in the preload section?

 

[BloodSoul]

Those loops triggers might as well be one trigger as opposed to two, also what's the purpose of the dummy which is created in the preload section?

I've made 2 triggers, because I thought they could "lag" the game if both of them were in same trigger. And about the dummy part: When you learn a spell for the first time, it might cause little lag. So, I did that to "remove" the lag. If it is useless, tell me, so I can remove it.

 

[Tank-Commander]

Why would it cause a little lag?

 

[BloodSoul]

Why would it cause a little lag?

It's something like this. But I just read this and I will remove it.

 

[Magtheridon96]

They wouldn't lag the game if they were in the same trigger :|
Players would be more susceptible to lag if you would split them up.

 

[xorkatoss]

awesome spell, MUI and leak-less gj 5/5 +rep

 

[Tank-Commander]

You should also be using Co-ordinates to move units as opposed to points, if you make the AOE bigger and put in a lot of units, you'll see why I say this - it'd lag like hell

 

[BloodSoul]

You should also be using Co-ordinates to move units as opposed to points, if you make the AOE bigger and put in a lot of units, you'll see why I say this - it'd lag like hell

Done. Thanks!

awesome spell, MUI and leak-less gj 5/5 +rep

Thanks!

 

[Thor]

awesome spell 5/5 + rep