23.12.2014, 16:39
This simple include could be a lot better :
1) format function is useless to copy a string, use strcpy instead.
2) If you're using YSI library for using y_hooks, why don't you use y_iterate too since there's a player loop?
3) Why a function named "GetServerMaxPlayers" when there's already GetMaxPlayers?
4) Server port doesn't change while running, so you can firstly store it on a variable and then keep on returning it when the function is used.
5) "GetServerMaxNPC" - Similar to the above statements.
1) format function is useless to copy a string, use strcpy instead.
2) If you're using YSI library for using y_hooks, why don't you use y_iterate too since there's a player loop?
3) Why a function named "GetServerMaxPlayers" when there's already GetMaxPlayers?
4) Server port doesn't change while running, so you can firstly store it on a variable and then keep on returning it when the function is used.
5) "GetServerMaxNPC" - Similar to the above statements.