[mamorunl]

Instead of these lines:

pawn Code:

if(i == (MAX_PLAYERS - 1))
        {
            if(strlen(fstr)) SendClientMessage(playerid, 0x9BBDDEAA, fstr);
        }

You could've just put it right outside the for loop and check the length (strlen) of the fstr string and if it is higher than 0, just print it. It saves you the weird if-statement.