[Exhibit]

Quote:

Originally Posted by RogueDrifter U should change this: Code: for(new i=0; i < MAX_PLAYERS; i++) { GetPlayerName(i, name, MAX_PLAYER_NAME); if(!IsPlayerConnected(i)) continue; To this: Code: for(new i=0; i < MAX_PLAYERS; i++) { if(!IsPlayerConnected(i)) continue; GetPlayerName(i, name, MAX_PLAYER_NAME); Also do you realize that this: Code: for(new i=0; i < MAX_PLAYERS; i++) { GetPlayerName(i, name, MAX_PLAYER_NAME); if(!IsPlayerConnected(i)) continue; SendDC(CHANNEL_ID, "```%s(%d)```", name, i); } Is going to send 500 discord messages if the server is busy enough? I'd suggest grouping it all up in one string and sending one message or at least do so by checking the amount of online players? One last thing is this: Code: GetPlayerName(giveplayerid, giveplayer, MAX_PLAYER_NAME); It is for the best always using the sizeof the varible you're storing the name in regardless if MAX_PLAYER_NAME is the same just for compatibility cautiousness in this case and all other cases seeing as if the size of that variable's array ever changes it will lead into problems. Nevertheless, there isn't much to look at seeing as its a small script, it's a cool idea lots of the effort done was put into the include&plugin that's for sure but it's different so good job to you, Inn0cent and maddinat0r.

I appreciate you pointing out my mistakes, It is always better to learn.