[feature request] mp_hegrenade_penetration should be renamed

[rust2]

as written in game.cfg:

// Disable grenade damage through walls
// 0 - disabled
// 1 - enabled
//
// Default value: "0"
mp_hegrenade_penetration 0

IMHO cVar name should be changed to mp_hegrenade_penetration_disable or something like that, and it is confusing me because when i see mp_hegrenade_penetration 1 I think grenades will deal damage to me. But it is opposite

[ShadowsAdi]

That is pretty clear. Disable grenade damage through walls ?

0 - function disabled
1 - function enabled

[rust2]

That is pretty clear.

I learned it only when checked cfg.

But cvar name should be self explaining IMHO.

Like if mp_hegrenade_penetration 1 then grenade will penetrate, if mp_hegrenade_penetration 0 then not. Seems logical for me.

[StevenKal]

Yes, CVar name is illogical. Feature in-game should be swapped if name is kept.

[Giferns]

bad idea to touch it as it will break plugins that use this cvar

[StevenKal]

Is really there plugins relying on it (like for a custom damage radius code)? In such case renaming the CVar will also break such plugins. In overal, changing just its internal behavior (swap) will break more the gameplay behavior until the admins change the CVar to the other value.