Spell Blink Back

[Sevion]

Why is it a Filter() call? You can just write the function there and it would be the same

Because it takes a [ljass]boolexpr[/ljass] not a [ljass]code[/ljass].

 

[WolfieeifloW]

Could a moderator review this.
Sim said I'd just need to fix the TimerUtils, which has been done (And more ).

Or, any comments on the code/spell itself?

 

[BlackRose]

[LJASS]call TimerStart(d.t, 1.233, false, function SFXDestroy)[/LJASS], effect duration should be configurable.

You also don't null "u" in BBConditions. I also don't think the BB prefix is necessary, since you already use scope and private.

 

[WolfieeifloW]

Fixed and fixed, thanks for your comment!

Anyone else, or a mod ?

 

[WolfieeifloW]

I was only supposed to update TimerUtils for this to most likely get re-approved;
Could a moderator, please and thank you, look at this

 

[BlackRose]

Functions listed under the below header should be constant so it is inlineable.

//=========================================================================
//====================     Functions Configuration     ====================
//=========================================================================

I see nothing else wrong other than the unconstant functions. The only other thing I would make note of is the code structure. I'd make any functions related with the spell only as methods to the struct, but it's not too big of a deal in this case.