Chromatic TimerUtils 2.0 [for 1.24b+]

[Vexorian]

Well, this one is suitable for all situations, hence I like it so I submitted it.

[PandaMine]

Originally Posted by Vexorian

Well, this one is suitable for all situations, hence I like it so I submitted it.

and the other one?

[Matrix_UA]

So GetCSData sucks?

And what is the best for attaching structs to units/items?

[Vexorian]

So GetCSData sucks?

That's news to me.

[Here-b-Trollz]

You could unconstantify 'MIN_HANDLE_INDEX' or whatever it is, then at map init loop through, create all your timers, set MIN_HANDLE_INDEX to the H2I of the first one and you wouldn't have to worry about size issues.

I think Toadcop does this?

[Vexorian]

Someone can do that and keep the XXXTimerXXX naming, that's the plan, I think I mentioned so in the first post. But I won't add it to the official version for a simple reason, loading 1000 timers at init is usually worse for robustness and performance than this, it needs a user that knows what he is doing.

[Anitarf]

Well, considering that I annoyed Vex into coding this I should write something about how awesome it is now that he did code it. Well, it is great and I intend to use it in all my future code that will need timers.

[cohadar]

I think we have more Timer "Unitls" on this forum than all other resources together....

[Vexorian]

Originally Posted by cohadar

I think we have more Timer "Unitls" on this forum than all other resources together....

I don't think we have any of those.

[cohadar]

OT:
Vexorian uses his admin privielges to fuckup my spelilng so he can make joeks about it.

[Vexorian]

Yeah sure, that's the most logical explanation.

[Troll-Brain]

Really minor suggestions :

JASS:

    function NewTimer takes nothing returns timer
        if (tN==0) then
            return CreateTimer() // yeah, we win a set :p
        else
            set tN=tN-1
        endif
        call SetTimerData(tT[tN],0)
     return tT[tN]
    endfunction

    //==========================================================================================
    function ReleaseTimer takes timer t returns nothing
        if(t==null) then
            debug call BJDebugMsg("Warning: attempt to release a null timer")
            return
        endif
        if (tN==8191) then
            debug call BJDebugMsg("Warning: Timer stack is full, destroying timer!!")

            //stack is full, the map already has much more troubles than the chance of bug
            call DestroyTimer(t)
        else
            if(GetTimerData(t)==HELD) then
                debug call BJDebugMsg("Warning: ReleaseTimer: Double free!")
                return
            endif
            call PauseTimer(t) // if the timer is already released, then we don't need to pause it again.
            call SetTimerData(t,HELD)
            set tT[tN]=t
            set tN=tN+1
        endif    
    endfunction

I love the simplicity of this library, i will use it for dummies units and group as well, gj.

[Vexorian]

Update.

Hmnn, Troll-Brain, didn't see your post till now, will take a look to those changes later.

[Toadcop]

You could unconstantify 'MIN_HANDLE_INDEX' or whatever it is, then at map init loop through, create all your timers, set MIN_HANDLE_INDEX to the H2I of the first one and you wouldn't have to worry about size issues.

I think Toadcop does this?

the main feature in my case. is what i align handle indexes before intializing timers.

[Vexorian]

update, err, fixed a thread crash in redInit...