[Jonny5]

you must format a string BEFORE you use it in SendClientMessage

you cannot use SendClientMessage to format anything.

pawn Code:

CMD:givecar(playerid,params[])
{
    new vehid;
    new id,str[50];
    new Float:X, Float:Y, Float:Z, Float:angle;
    if(sscanf(params,"du",vehid, id)) return SendClientMessage(playerid,-1,"Error: - /givecar [vehid] [player id]");
    GetPlayerPos(id, X, Y, Z);
    GetPlayerFacingAngle(id, angle);
    CreateVehicle(vehid, X, Y, Z, angle, 0, 1, -1);
    PutPlayerInVehicle(id, vehid, 0);
    format(str,sizeof(str),"%s has given you a vehicle", GetPlayerName(playerid));
    SendClientMessage(playerid, -1, str);
    return 1;
}