- Joined
- Jul 10, 2015
- Messages
- 22
amazing
-
Are you planning to upload your awesome spell or system to Hive? Please review the rules here.
-
🏆 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!
Master Spell Reflector
- Author(s)
- KitsuneTailsPrower
- Size
- 112.31 KB
- Rating
- Downloads
- 48
- Created
- Jan 15, 2024
- Updated
- Jan 15, 2024
- Resources
- 1
- State
Awaiting update
This bundle is marked as awaiting update. A staff member has requested changes to it before it can be approved.
A template for making a Spell Reflector that works pleasingly for both Missile and Instant impact spells. Requires some JASS knowledge to use fully.
Contents
Reviews
Spell Reviewer
- Joined
- Nov 18, 2012
- Messages
- 1,892
The configurable functions should be next to IsSpellInstant instead of being spread around. The indention is kind of messed up, sometimes too many spaces, some times inconsistent space where it should be.
You could also cache GetTriggerUnit() and the rest, specially when the lines are quite long, not only it is faster (in terms of speed) but also clearer to see and you don't risk surpassing the character limit on a line. This one is only a minor problem.
This template enables pretty good possiblities to explore different types of spell reflections. It's almost good for approval.
You could also cache GetTriggerUnit() and the rest, specially when the lines are quite long, not only it is faster (in terms of speed) but also clearer to see and you don't risk surpassing the character limit on a line. This one is only a minor problem.
This template enables pretty good possiblities to explore different types of spell reflections. It's almost good for approval.
