View Single Post
RedSword
SourceMod Plugin Approver
Join Date: Mar 2006
Location: Quebec, Canada
Old 03-03-2015 , 14:35   Re: [NMRiH] Laser Aim
Reply With Quote #7

Hey, I highly suggest that you put FCVAR_DONTRECORD back; if you're to update your plugin people will still have the previous version number in their .cfg if you don't have FCVAR_DONTRECORD for your version CVar. You also need FCVAR_NOTIFY for your version CVar (needed for approval).

It's a good thing your changed your version CVar description :$ (I didn't realize the previous one).

You don't necessarily have to put limits on every CVar; I suggest only using them when the limit must be respected or to give information to the user (i.e. if some user would like to try having "red" over 255, they would be a bit confused seeing it has no effects or confusing effects). I.e. for sm_laser_aim_dot_width users might want to play with it; there are no real problem if it is not respected. Also you can put a minimum without a maximum.

On a side note, I believe you could send your TE_ to the only user concerned (i.e. you could add an option to prevent seeing others TE)

Only first paragraph is regarding approval.

Red
__________________
My plugins :
Red Maze
Afk Bomb
RAWR (per player/rounds Awp Restrict.)
Kill Assist
Be Medic

You can also Donate if you appreciate my work
RedSword is offline