Booster bug?

[Dersaidin]

You walk up to a booster, you get poison. The poison lasts for 30 seconds.

The problem is, if you go back to the booster with 10 seconds of poison left, touch it... you still only have 10 seconds left. You should get another thirty (not 10 + 30, just back to 30.)

[Odin]

I told this to tjw in a match, and I don't think he understood me.

[Delvin]

Second that. It's annoying when you have to wait until the poison runs out if you want to leave with the full effect.

[techhead]

Add another signature to the petition.
Anyone out-there against this idea?

[rasz_pl]

its intentional

rasz@capek:/media/hdc3/home/rasz/source/trunk1/src/game$ cat tremulous.h  | grep BOOS
#define BOOSTER_BP                  12
#define BOOSTER_BT                  15000
#define BOOSTER_HEALTH              ABHM(150)
#define BOOSTER_REGEN               8
#define BOOSTER_SPLASHDAMAGE        50
#define BOOSTER_SPLASHRADIUS        50
#define BOOSTER_CREEPSIZE           120
#define BOOSTER_INTERVAL            30000 //time in msec between uses (per player)
#define BOOSTER_REGEN_MOD           2.0f
#define BOOST_TIME                  30000

I can submit a lame patch changing
#define BOOSTER_INTERVAL            30000 //time in msec between uses (per player)
to
#define BOOSTER_INTERVAL            1 //time in msec between uses (per player)

but I doubt it will go thru

[Dersaidin]

#define BOOSTER_INTERVAL 1000
would make more sense. No need to use it every single moment.

[Undeference]

BOOSTER_INTERVAL only actually occurs twice. It is defined, then we have:

Code Select Expand

 //only allow boostage once every 30 seconds
 if( client->lastBoostedTime + BOOSTER_INTERVAL > level.time )
   return;in ABooster_Touch in g_buildable.c.
Is it really that important one way or another? If it is, maybe BOOSTER_INTERVAL should actually be higher, maybe like 1 minute. Otherwise, maybe it should be removed entirely or turned into a cvar people can screw with.

(Why cat|grep when grep will read the file for you? )