• 🏆 Texturing Contest #33 is OPEN! Contestants must re-texture a SD unit model found in-game (Warcraft 3 Classic), recreating the unit into a peaceful NPC version. 🔗Click here to enter!
  • It's time for the first HD Modeling Contest of 2024. Join the theme discussion for Hive's HD Modeling Contest #6! Click here to post your idea!

Armargeddon V1.1

This bundle is marked as useful / simple. Simplicity is bliss, low effort and/or may contain minor bugs.
This is one of my first GUI spells i have checked it for lecks.
it causes lightnin and meteors to strike down all units within the area your unit is free to move around wance activated.

Change Log:
V1.1
-Remove custom models
-changed armagaddon spelling
-made triggering easyer

Keywords:
Spell, Fire, Ice, Lightning, Armorgedon, Kill
Contents

Just another Warcraft III map (Map)

Reviews
08:04, 2nd Jul 2010 Hanky: Sorry but your spell is too simple.

Moderator

M

Moderator

08:04, 2nd Jul 2010
Hanky:
Sorry but your spell is too simple.
 
You can't use imported resources for spells. Your going to want to change them out for in game ones.

Looking at coding now......

Review:
Well it's leakless, but it's not exactly a good spell.
It's essentially just starfall with changed models and a dummy unit casting it - that's not exactly something that someone would need to look for a spell here to do.

You used: "If owner of casting unit = player 1"
Why? This would make this only work for player 1, this spell is MUI but doing that makes it only work for 1 player - that if then is unnecessary.
The "Dummy unit" variables is unnecessary - just order the last created unit to cast it, that won't leak.
Things that do leak that you need to worry about removing (in general, not in your spell) are locations and groups.


Your main problem is your over complicating something simple and easy to do.

Sorry but this is without a doubt going to be rejected


Also you misspelled Armageddon
 
Top