Crowbar uses incorrect time base for nextthink value

[SamVanheer]

When the Crowbar is swung and something is hit a delayed response is created using a think function. Unfortunately the code was incorrectly updated when weapon prediction was implemented:

halflife/dlls/crowbar.cpp

Line 310 in 5d76170

pev->nextthink = UTIL_WeaponTimeBase() + 0.2;

This should be using gpGlobals->time.

[JulianHeuser]

Just wondering, what's the consequences of this in-game? Would this fix the rapid crowbar bug?

[SamVanheer]

No this doesn't affect that, #617 tracks that bug.

The consequences of this are that the next think time is in the past and so will run more quickly than intended. For correctness the time should be at some point >= current time. It may also be responsible for occasional longer delays between hits, i'm not sure about that.

[mikela-valve]

Fixed, will be included in next beta update.

[mikela-valve]

Fixed in beta 'Exe build: 11:12:36 May 21 2019 (8244)'.

[mikela-valve]

Closing as fixed.