Removing cvars is never a good option unless it somehow decreases performance drastically like if it was used in prethink.
also, what is this:
Code:
public s1(id,level,cid){
client_cmd(id, "weapon_p228");
client_cmd(id, "weapon_scout");
client_cmd(id, "weapon_xm1014");
client_cmd(id, "weapon_mac10");
client_cmd(id, "weapon_aug");
client_cmd(id, "weapon_ump45");
client_cmd(id, "weapon_sg550");
client_cmd(id, "weapon_galil");
client_cmd(id, "weapon_famas");
client_cmd(id, "weapon_awp");
client_cmd(id, "weapon_mp5navy");
client_cmd(id, "weapon_m249");
client_cmd(id, "weapon_m3");
client_cmd(id, "weapon_m4a1");
client_cmd(id, "weapon_tmp");
client_cmd(id, "weapon_g3sg1");
client_cmd(id, "weapon_sg552");
client_cmd(id, "weapon_ak47");
client_cmd(id, "weapon_p90");
}
I guess this doesn't trigger the weapon switching hud. But Isn't there another way of doing this? If not, then at least make it cleaner and put those weapons into an array and loop through them. Don't know if it has any benefit performance wise, but the code will look a lot cleaner and more readable. At least for me.
Long ago I have found this and actually used it.
https://forums.alliedmods.net/showthread.php?p=680121 Doesn't it have the same functionality, but better?