Windcutter v 1.5

[Moderator]

15:48, 24th Feb 2011
The_Reborn_Devil:

My review


Status: Approved
Rating: Useful

 

[Weep]

Neat effect, though the blood splashes are too bold; maybe use a different blood model and create them less frequently?

I haven't reviewed the code.

 

[Marsal]

now it looks a little differ from the screen because some reason it don't let me upload a new screenshot :S
edit: oh finally

 

[RaijinSpark]

Basing from the screenshot given, the effects is cool and the spell is overall awesome and unique.

Downloaded, I'll check the triggers later.

 

[Noobsmoke]

it is not recycling right ?

 

[Marsal]

it does, when it reaches the maximum array value it starts again from one, if you worry about that it could exceed an other running instance then just increase the WMax in the init trigger

 

[Noobsmoke]

that's not the best recycling, becose if you cast it without pause it will be laggy

 

[Marsal]

I know it's not the best but hey, it works and now the only lagg causing thing is that I used 24 dummy per cast ~.~ 6 blade and 18 wind dummies
reduced it to 12 per cast

 

[Marsal]

hey if you think the spell is okay then pls vote for approval ~.~

 

[ShiroiSenji]

Here to back you up, marsal, spell looked good. I think ill use it at shonen wars.

+Rep
+Approval

 

[The Reborn Devil]

Moderator review:
The triggering looks ok, but there's no need to destroy WDummyGroup. Just clear it. Your recycling could be better too, but it's ok. Btw, in the loop in your Init trigger, just do this:

• Custom script: set udg_WDummyGroup[GetForLoopIndexA()] = CreateGroup()

More efficient ^^

 

[Marsal]

Thanks, I'll modify some of these little things ~.~

 

[baassee]

In the Init trigger the Integer A loop should loop from 1 to WMax

As said the indexing is kind of meh, looping 100 times even if only one instance is on. And yes I know that you can lower that value but then the safety will be reduced too when you think about it.

It should be mentioned that the damage value is damaged to the units for 1/0.03 times every second.

Else it looks good enough to get approved I guess.

 

[Marsal]

In the Init trigger the Integer A loop should loop from 1 to WMax

As said the indexing is kind of meh, looping 100 times even if only one instance is on. And yes I know that you can lower that value but then the safety will be reduced too when you think about it.

It should be mentioned that the damage value is damaged to the units for 1/0.03 times every second.

Else it looks good enough to get approved I guess.

Heh I was a little lazy ~.~ My other spells counts to the max.

It's easy to calculate what number would be the best for a map.

The damage is okay, I think. Now it deals the damage constantly and it looks nice.

 

[Marcos DAB]

Very impressive.

 

[Terror]

nice xD

 

[Marsal]

New version. Got rid the arrayed TempPoint variable and changes a part of the system mentioned at other spells.

 

[Black-kitty]

Wo that looks really cool, good job.

 

[Rip_and_Tear]

very nice spell and very cool effect. +rep

 

[Peacehagen]

I was just wondering if this spell works if more than one Hero casts it at the same time? Thanks for any response

 

[Marsal]

Of course. If not, it wouldn't be approved ~.~

 

[Peacehagen]

Oh, ok, great

 

[Maximum_Evil]

awesome effect, but the damage is way too high and not everyone knows how to decrease it

 

[Marsal]

jk right? in the init you just have to modify the damage variables' values....

 

[crystal.maiden_16]

Awesome spell, I'm going to use it for wind-themed hero...
+rep