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.
|
1. The conversion of fakemeta money was in the first plugin and I was a little bit busy to convert all the functions, but don't worry will convert them soon!
2. Right! Will do in the next version
3. You have a point, will remove them when the plugin will reach it's final stages.
4. That was an idea, I wanted where the player shot the nanosuit armor to show a spark, but I forgot to remove it, anyway it's a stock so it will disapear.
5. For that I know that there is a define but I know that it doesn't work.
6. Will add them to the ML system.
7. Well never thought about that, will do that ;).
And the last thing is that I will make my plugin use my
Speed API.
Thanks for pointing all these things out.
Edit: Updated it will almost all things that you requested!
__________________