HomeUser Control Panel (unavailable in archive)ForumsTutorialsArt GalleryResourcesMaps

FireElemental and MinedMissile

02-12-2006, 11:33 AM#1
sheep.spirit
2 spells i wrote for my AOS map TEAR, Fire elemental leaves foot prints behind that damages and shoots fire balls at enemies. Mined missile is a missile with mines that may drop off. Credit if use. Uses the caster system. And pls tell me of any leaks or bugs you can find, cause i am not sure whether i removed all the trigger actions or not. ( I think i removed all... but still...) thanks


Done!!, version 1.1 fixed lots of minor bugs with description and fixed mined missile code, (looks nicer now)

Update: v1.2 fixed a stupid typo bug which screwed up fire walk... Update recomended
Attached Images
File type: jpgfire.jpg (68.9 KB)
Attached Files
File type: w3xFireElementalandMinedMissile.w3x (117.3 KB)
02-12-2006, 12:12 PM#2
Vexorian
You should have a trigger that revives the heroes or at least give them an ankh of reincarnation or something.

Can't approve it because it has location leaks.

Like this line:
Collapse JASS:
     call SetUnitPositionLoc( e, PolarProjectionBJ(GetUnitLoc(e), 15.00, GetUnitFacing(e))) 

Get A newest caster system version that comes with CS_Error so you can use it instead of SimError (makes implementing easier)

Just fix that memory leak and make sure there aren't other memory leaks. And I 'll be able to approve it
02-12-2006, 09:50 PM#3
sheep.spirit
Yeah, i only thought of that when i was in bed, too lazy to change it, but now its fixed, cant find anymore location leaks other than the line you pointed out. type -revive many times to revive hero, cause there is a little bug.
02-13-2006, 02:01 AM#4
Vexorian
hmnn Yes. I have more problems. Some are really minor.

Since the adition of this new version of the Caster System, you have to change those >= 12.7 in the comments with 13.1 maybe.

Also the new url is http://wc3campaigns.net/vexorian/

Those were minor problems , there is something that troubles me and it is your Mined Missile ability. It shouldn't use that dummy unit at all. I think you could make it much better without relying on creating dummy units and using the enter rect events. Use tables and timers, the code would look much better and you would not need that unit for no reason.

I am approving them anyways
02-13-2006, 07:30 AM#5
sheep.spirit
Yep, i thought so too, cause the miendmissile ability was made long ago, beofre i knoew about the trigger add action thing, so i think i am rmking the mined missile with more effeciency and better coding. I may release it with my next spell pack
02-25-2006, 10:10 PM#6
Blade.dk
They seems fine to me - but the fireballs that the lava spawn creates seems to always go to the same direction, I think it would be better if they just moved towards close enemies?