[Fixed] Delay ... waiting forever

[kn]

If you pass 0 as an argument to delay(), the script will not continue:

script 1 OPEN
{
delay(3 * 35);
print(s: "First run .... ");
delay(2 * 35);
ACS_Execute(2,0,90,0,0);
scriptwait(2);
delay(2 * 35);
print(s: "second run .... delay bug .. waiting forever");
delay(2 * 35);
ACS_Execute(2,0,0,0,0); // BUG!!!!!! change it to 2,0,1,0,0 and everything is okay
scriptwait(2);
Exit_normal(0);
}

script 2 (int dtime)
{
Print(s: "you passed ",d:dtime,s:" to this script");
Delay(dtime);
Print(s: "OK");
}

[Cyb]

why do you need a delay of 0? just leave it out :P (alternativly 1 tic is hardly a noticable delay, so using 1 is just as effective, delay(dtime + 1);

[Graf Zahl]

why do you need a delay of 0? just leave it out (alternativly 1 tic is hardly a noticable delay, so using 1 is just as effective, delay(dtime + 1);

... or change it to 'if (dtime>0) delay(dtime);'


Anyway, a delay of length 0 should not wait. As it is the 0 gets decremented without being checked and the script will wait a very long time (almost 4 years) until it continues...

[Hirogen2]

Anyway, a delay of length 0 should not wait. As it is the 0 gets decremented without being checked and the script will wait a very long time (almost 4 years) until it continues...

Sounds like a "unsigned long dt" in the ZDoom code, which hehfully decrements from 2^32 -1.

[kn]

why do you need a delay of 0? just leave it out (alternativly 1 tic is hardly a noticable delay, so using 1 is just as effective, delay(dtime + 1);

I don't need it. But ... I assigned ACS_Execute() to a "player enters sector" thing and I forgot to change the value.

I just noticed that this script didn't finish. I expected a delay(0) to immediately skip to the next command instead of waiting till the end of days.

[kn]

Anyway, a delay of length 0 should not wait. As it is the 0 gets decremented without being checked and the script will wait a very long time (almost 4 years) until it continues...

Hey. that's great.

Hmm.. this script doens't continue .. I'll be back in 4 years. [/quote]

[randi]

Fixed for 2.0.48/2.1.0.