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

Hey Grey83,

The plugin looks fine and I'll approve it if you remove your FCVAR_REPLICATED, as it may cause error for clients.

A few things you could also change (optional) :
  • You may want to limit R/G/B/A values to 0-255 (SM allows to set up easily such restrictions on a CVar)
  • When a client is IsClientInGame, he is necessarily Connected, so not need to check if he is connected.
  • When you get so many comparison, you may prefer to put all the weapons you want the beam for in a Trie, that way you can compare it in O(1)

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