Prevent Cast

[PurgeandFire]

EDIT: Damn, gy this. As cool and short as it was, I just realized that PreventOrders does this too. And also, it prevents it right at the issue, not on the cast. (eg: if you cast something point target far away where you need to move to it)

 

[Nestharus]

This has quite the overhead
struct CastList

Try using an array struct instead.

Also, you should be using a hashtable + unit indexing instead of a linked list to prevent the casting... a linked list is just a really poor choice for this >.>.

Also, these should all be private

        thistype next
        thistype prev
        unit caster
        integer abilID
        real cp

And you should have all of your stuff inside the struct. No reason any of it should be outside.

It'd also be neato to have a prevent cast event =).

This might as well go into a module initializer

    private function Init takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterAnyUnitEventBJ(t,EVENT_PLAYER_UNIT_SPELL_CAST)
        call TriggerAddCondition(t,Condition(function onCast))
    endfunction

>.>.

Just try to make it a habit of only using module initializers unless you need your thingie to initialize later =D.

This is bad
call IssueImmediateOrder(c,"stop")

Some abilities don't stop a unit (they keep moving) like berserk.

 

[PurgeandFire]

Damn. PreventOrders does this lol. I thought it was only for unit moving orders and attacks... Oh well, at least it didn't take super long to make.

 

[Bribe]

Better late than never; graveyarded.