View Single Post
FeuerSturm
AlliedModders Donor
Join Date: Apr 2004
Old 12-13-2008 , 14:20   Re: DOD:S Reverse Tag
Reply With Quote #9

Quote:
Originally Posted by FeuerSturm
- not rely on the funcommands plugin (sm_beacon) as it might not be loaded
the command sm_beacon that you're using is part of the "funcommands"
plugin that comes with SourceMod and a lot of people (including myself)
aren't using/loading that plugin at all, so you loose half of the functionality
of you plugin.
You could rather take a look at the beacon part and adapt it to your needs
("borrow" some code of it and use it in your plugin).
Either that or clearly state that the funcommands plugin is obligatory
for your plugin to work.

Quote:
Originally Posted by FeuerSturm
- kill the repeating timer if the plugin is off and restart it once it is enabled
You are using a repeating timer that calls your function every 2 seconds
from plugin start on no matter if it's needed or not.
You should only start it once it is needed and end it once it is not anymore.

Quote:
Originally Posted by FeuerSturm
- add a global tracking convar (optional)
exactly what Lebson506th said!


P.S.:
If you're about to create more DoD:S specific plugins, feel free to head over to
DoDSP Community and apply to join the Developer group, you'll be able to
ask for scripting help and review before publically posting your plugin here then.

Last edited by FeuerSturm; 12-13-2008 at 14:23. Reason: added P.S.!
FeuerSturm is offline