Fire Impact v1.3

[Moderator]

12:45, 13ndth Sep 2015

This resource has been re-approved by BPower,

Criticism:
I agree with Hanky, that the code could be written a lot more efficient.
IsTerrainWalkable is a better and more convenient snippet to check the walkability of a coordinate.
You should change to that version. The dummy scaling could be configurable as well.

From moderator to user:
Make sure you change the path-ability snippet.

previous moderation from 2008

20:10, 3rd Dec 2008
Hanky:
Well the spell is well made. The script is good but at some points it could be more effective. But well, your spell is leakless and MUI, so far I can see. The visual effects are ok but I suggest to add a spawn effect, something like flames which appear from the ground and shoot those fireballs. The documentation and tooltips are ok too.

Well all in all a good spell which I only can recommend to use if this spell is that what you are after.

Possible Improvements:
- script more effective

 

[Yixx]

A nice spell! Fireballs flying around, hitting units,,
But i dont really like the last huge fire blast,, it makes it look a little overpowered, and after,, al units are dead anyway! (or is that because it is peasant?)
I dont know anything about Jass (yet hopefully ^^ ), so i wont say anything about the script,,

Conclusion: I like it! Good spell.. 4/5

 

[TheBlooddancer]

Would be cooler, if it didnt have the overpowered SPLASH at the end.
Also, it makes you unable to see that the fireball bounces.

 

[xD.Schurke]

hehe can change it, it's no problem , but have to find a better way then i will look what i can do

 

[TheBlooddancer]

Well, it will be a whole new spell, then. Anyhow~~

 

[xD.Schurke]

i also found a bug in the script (mini missiles didn't deal damage), and forgot to set that firebolts collide with walls etc. will fix it and upload new version

 

[Dynasti]

Good idea and somewhat good efficiency, but some parts of the script i find alot of things that could need improvements.. but all in all a good spell 4/5

Edit: /!\ Leak Found! /!\ in the fireCond you leak at local variable loc ^^

 

[TheBlooddancer]

It's really shit now. When i cast it in one direction, it makes it a whole another.

Just.. Fix it D:

 

[xD.Schurke]

It's really shit now. When i cast it in one direction, it makes it a whole another.

Just.. Fix it D:

fixed it, removed one thing one the wrong line xd

 

[TheBlooddancer]

Meh.

Make one, where he just shoots a fireball in that direction, which bounces in that direction.

 

[StaberFire]

The visuals are nice, and the bouncing effect is a great touch as well. I didn't look over the code 'cause I'm a self-proclaimed JASS noob, but I didn't feel like it lagged at all.

Over all nice job, but I can think of a better name for the spell:
Peasant Cleaner XD

 

[kewlthis]

The effect is bad, and fireballs seem really random, in a bad way.

 

[aaron]

i think this spell is exceptional but the leaks wont be hard to fix just need to specify the variables a little better.

 

[xD.Schurke]

i think this spell is exceptional but the leaks wont be hard to fix just need to specify the variables a little better.

there are no leaks

 

[Rex.]

Nice.