Tremulous Forum

General => Feedback => Topic started by: Dersaidin on November 12, 2006, 01:56:50 am

Title: Booster bug?
Post by: Dersaidin on November 12, 2006, 01:56:50 am

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.)

Title: Booster bug?
Post by: Odin on November 12, 2006, 03:25:19 am

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

Title: Booster bug?
Post by: Delvin on November 12, 2006, 12:43:14 pm

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

Title: Booster bug?
Post by: techhead on November 12, 2006, 07:35:01 pm

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

Title: Booster bug?
Post by: rasz_pl on November 12, 2006, 08:24:26 pm

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

Title: Booster bug?
Post by: Dersaidin on November 12, 2006, 09:31:54 pm

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

Title: Booster bug?
Post by: Undeference on November 13, 2006, 07:08:41 am

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

Code: [Select]

 //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? :P)