Quote:
Originally Posted by arkshine
- If using cstrike, why did you bother to convert in fakemeta the money ? ( or reversely, why did you convert only the money ? )
- colored_msg() and fm_set_user_money() ; the value of get_user_msgid() should be saved just one time.
- 'stock' label are pointless here.
- TE_SPARKS : use MSG_PVS. No need to show sparks to players who don't look to this origin. ( better )
- "You are not in a buy zone." I know that there is a define text for that like the others but I don't remind.
- You forget to add to the ML 2 messages : ""You have bought a nanosuit armor!" and "You don't have a nanosuit!".
- Since is_user_bot() is used severals time, you may want to optimize by saving one time the type on 'putinserver' forward.
That's all for me, for the moment. Feel free to ignore or not.  |
Stock labels are never pointless. Why in hell would you include a function that will not be called, and cannot be called from other plugins? Having to comment a function or remove it is just waste of time