[Chriham3]

ahem...

if(strcmp(cmdtext, "/makecop", true) == 0)
{
new tmp[128];
new player1 = strval(tmp);
if(IsPlayerConnected(player1))
{
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_GRAD1, "USAGE: /makecop [playerid]");
}
if (!IsPlayerConnected(player1)) return SendClientMessage(playerid, color, "player is not connected.");
{
CopMake(player1);
SendClientMessage(player1, COLOR_BLUE, "You have been made a police officer. Changes will appear on next death.");
}
}
} // where did that one come from?
return 1;
}


Thanks, but you added one closing bracket to much.