Quote:
Originally Posted by hornet
@Pastout I've taken a look through your plugin and taken a few notes, I've hardly read through in detail, but its what was most easily noticeable to me. I know that this has been around for a long time, but regardless, here's some things that you can work towards to make your plugin more efficient:
|
Things Done IN
BOLD:
-Disable FM_PlayerPreThink when not in use - only used during certain events.
-Disable FM_AddToFullPack when not in use - only used in certain events.
-Disable FM_Think when not in use.
-Disable Ham_Touch when not in use.
-Should be consistent with alive checks - since you have a bitsum for alive players you should never have to use is_user_alive().
-StatusIcon method used to prevent players from using buyzones is outdated and inefficient. Use m_iBuying with info_map_parameters.
As For this I really do not see the need for this, but if you want to see it done just let me know...
-Remove FM_Touch. Use Ham_Touch and disable it when not in use ( or engine touch would be acceptable if applicable ).
-FM_Voice_SetClientListening forward would benefit if you were to cache player's ability to speak based on their team.
ADD New Weather Commands:
Cvar set_weather normal(default)
For sunny
weather (set_weather) to either
For rainny weather (set_weather) to eitherFor snowy weather (set_weather) to eitherFor snowy/rainny weather (set_weather) to eitherCurrent Version
1.7.3