Thread: SpellEvent
View Single Post
Old 07-16-2009, 10:40 PM   #36
Anitarf
Procrastination Incarnate


Development Director
 
Join Date: Feb 2004
Posts: 8,190

Submissions (19)

Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)Anitarf has a brilliant future (903)

2008 Spell olympics - Fire - SilverApproved Map: Old School Alliance TacticsHero Contest #2 - 3rd PlaceSpell making session 2 winner

Default

The benefit of having all spell event responses available on all spell events is significant, the speed gained by removing it is not.

I could write another paragraph on why the feature shouldn't be optional, but since the speed gain is not significant in the first place, I don't see a reason to spend more time explaining this.

Besides, the bj_wantDestroyGroup-esque syntax is ugly.

If your particular map doesn't require this feature then you can always delete the event responses code from the library yourself, nobody is forcing you to only use the code as provided.

The second optimization you suggested is meaningless, an if statement isn't faster than an exitwhen statement in any meaningful way.
__________________
Anitarf is offline   Reply With Quote