[nikhilgupta345]

Originally Posted by ConnorMcLeod RegisterCVars function doesn't need to be public g_szWeaponClassnames shouldn't be global, declare it in RegisterCVars function so memory is free when you exit the function. i < MAX_WEAPON + 1 i <= MAX_WEAPON would make more sense Little tip : PHP Code:             formatex( szCVarName, charsmax( szCVarName ), "max_distance_%s", g_szWeaponClassnames[ i ] );                          replace( szCVarName, charsmax( szCVarName ), "weapon_", "" );  -> PHP Code:             formatex( szCVarName, charsmax( szCVarName ), "max_distance_%s", g_szWeaponClassnames[ i ][ 7 ] );  PHP Code:         static Float:flAttackerOrigin[ 3 ];         static Float:flVictimOrigin[ 3 ];  Static is not needed at all there.         if( !g_pWeaponCVARs[ iWeapon ] )             return HAM_IGNORED;                 new iMaxDistance = get_pcvar_num( g_pWeaponCVARs[ iWeapon ] ); You read 2 times the same array, would be better to cache pointer result. DMG_HEGRENADE check is useless, it will never happen.

Fixed.