Timer help

[Fingolfin]

Hi.
I have a function for adding extra time to a global timer; it looks like this:

function wanted takes player p, real decay returns nothing
    local real timeleft 
    set timeleft = TimerGetRemaining(udg_Player_WantedTimers[GetPlayerId(p)])
    call PauseTimer(udg_Player_WantedTimers[GetPlayerId(p)])
    call TimerStart(udg_Player_WantedTimers[GetPlayerId(p)], timeleft+decay, false, null) 
    //return
endfunction

And then a trigger that looks like this:

• WantExpire1
  • Events
    • Time - Player_WantedTimers[1] expires
  • Conditions
  • Actions
    • Player - Make Player 1 (Red) treat Player 2 (Blue) as an Neutral
    • Player - Make Player 1 (Red) treat Player 10 (Light Blue) as an Neutral
    • Player - Make Player 10 (Light Blue) treat Player 1 (Red) as an Neutral
    • Player - Make Player 2 (Blue) treat Player 1 (Red) as an Neutral

(there is a similar trigger for every player)

Thing is, it doesen't work. What am i doing wrong?

 

[Anachron]

Make sure the timers are created, so they are not null.
( set udg_timer[x] = CreateTimer() )

 

[Dr Super Good]

You never unpause the timer?
Also you can initialize locals to a value.
Thirdly make sure that you are using it only on player 2 blue (Player(1)) as that event will only work if you modify that timer not the timer for player 1 red (Player(0)).

 

[Fingolfin]

You never unpause the timer?
Also you can initialize locals to a value.
Thirdly make sure that you are using it only on player 2 blue (Player(1)) as that event will only work if you modify that timer not the timer for player 1 red (Player(0)).

I didn't really understand that..
Doesen't "TimerStart" automatically unpause the timer?

Though i tried to deal whith the undecleared timers by changing the function like this:

function wanted takes player p, real decay returns nothing
local real timeleft
if udg_Player_WantedTimers[GetPlayerId(p)] == null then
set udg_Player_WantedTimers[GetPlayerId(p)] = CreateTimer()
endif
call BJDebugMsg("TIMERS STARTING!")
set timeleft = TimerGetRemaining(udg_Player_WantedTimers[GetPlayerId(p)])
call PauseTimer(udg_Player_WantedTimers[GetPlayerId(p)])
call TimerStart(udg_Player_WantedTimers[GetPlayerId(p)], timeleft+decay, false, null)
//return
endfunction

 

[aznricepuff]

In JASS, red is Player(0), blue is Player(1), etc whereas in GUI red is Player(1), blue is Player(2), etc. Make sure you take this into account.

 

[Fingolfin]

In JASS, red is Player(0), blue is Player(1), etc whereas in GUI red is Player(1), blue is Player(2), etc. Make sure you take this into account.

Ah, yes!
Ofcourse, thank you.