Approved
Spell Reviewer
- Joined
- Nov 18, 2012
- Messages
- 1,894
You could cache GetSpellTargetUnit in onCast function into a local variable.
I don't understand the need to set the unit's ability over the unit's max mana. I tested and it seems the system works without it, is there any case for it?
Overall, it's a nice alternative implementation, the major difference is when one casts missile-type spells on the unit (storm bolt).
Approved
I don't understand the need to set the unit's ability over the unit's max mana. I tested and it seems the system works without it, is there any case for it?
Overall, it's a nice alternative implementation, the major difference is when one casts missile-type spells on the unit (storm bolt).
Approved
