Quote:
Originally Posted by shine771
No SteamID support yet.
|
Imho Steam comes first, but thats your choice.
Quote:
Originally Posted by shine771
I wont change anything about maxplayers. It's fine as it is.
|
Your choice to not do that, but it's a small change and may be worth it due to it being used in frequently called functions. Your should also store is_user_connected on putinserver, just seeing that.
Quote:
Originally Posted by shine771
Give me 3 examples.
|
- Everything that has to do with get_gametime in PlayerCmdStart.
- PlayerCmdStart could possibly be optimized to not create 2 traces not do all that origin stuff twice (is possible in worst case as I see it).
- In Handler_Think you get pev_origin twice, no reason to do that.
Quote:
Originally Posted by shine771
As i told you, i wasn't thinking with my head when i wrote it.
|
You should probably do that ;) But thats what we are here for.
Another thing I just spotted:
PHP Code:
new Name[33]
SQL_ReadResult(Query,1,Name,32)
You don't use the name so leave that out (maybe also change the query according to that).
Quote:
Originally Posted by shine771
Updated. Thanks  . |
No problem.
__________________