[bullet28]

Why?
From my experience, the chainsaw is one of the least popular weapons in-game.
In terms of weapon balance there is no reason for use-time (fuel) limitaion.

No cvars.

[Jerry_21]

hello bullet28, first of all excuse me for my english, I use a translator. You have just commented on a good idea that would be why not make chainsaws appear with greater probability on the maps with a cvar that everyone modifies to the probability that this weapon appears between maps. Greetings.

[BloodyBlade]

A shorter version:

[little_froy]

Originally Posted by BloodyBlade A shorter version:

it's not safe, you didn't check is the return value GetPlayerWeaponSlot(client, 1)valid and is a chainsaw.

[Silvers]

char classname[32];

could be

char classname[16];

since the classname itself is 15 chars long, no need for excessive bytes being nulled

and to optimize further make it "static char".

@BloodyBlade: checking if (event.GetInt("weaponid") == 20) { would optimize it further, just because it's shorter code doesn't make it more efficient.

weapon_fire event is triggered constantly when holding down the fire button, so should be optimized as best as possible to prevent unnecessary checks.

[BloodyBlade]

Originally Posted by little_froy it's not safe, you didn't check is the return value GetPlayerWeaponSlot(client, 1)valid and is a chainsaw.

If StrEqual(classname, "weapon_chainsaw") returns that the firing weapon is a chainsaw, then GetPlayerWeaponSlot(client, 1) returns a valid value. I have never received an error message with this code.

[BloodyBlade]

Originally Posted by Silvers char classname[32]; could be char classname[16]; since the classname itself is 15 chars long, no need for excessive bytes being nulled and to optimize further make it "static char". @BloodyBlade: checking if (event.GetInt("weaponid") == 20) { would optimize it further, just because it's shorter code doesn't make it more efficient. weapon_fire event is triggered constantly when holding down the fire button, so should be optimized as best as possible to prevent unnecessary checks.

Updated based on your comment.

[little_froy]

Originally Posted by BloodyBlade If StrEqual(classname, "weapon_chainsaw") returns that the firing weapon is a chainsaw, then GetPlayerWeaponSlot(client, 1) returns a valid value. I have never received an error message with this code.

weapon in which slot can be modified by addons, for example it can be in slot 0. and, other plugin can call api such as "SDKHooks_DropWeapon" to change current active weapon if the event hook earlier than you plugins.
extreme situations should be considered.

[BloodyBlade]

Originally Posted by little_froy weapon in which slot can be modified by addons, for example it can be in slot 0. and, other plugin can call api such as "SDKHooks_DropWeapon" to change current active weapon if the event hook earlier than you plugins. extreme situations should be considered.

I highly doubt that the player will decide to drop the weapon while cutting zombies.
If I see at least one error, I will change the code.