[Revo]

First of all, SCM is not a actual function, you need to define it first.

pawn Код:

#define SCM "SendClientMessage(%1, %2, %3);"

Don't quote me on the define, I never did it like that, I always use SendClientMessage(playerid, COLOR, Message); because I'm not lazy.

Loose identation means that you're not identing corrently (tabbing your code)
http://en.wikipedia.org/wiki/Indentation

pawn Код:

if( value == 1 ) {
    SendClientMessage(playerid, -1, "hi");
}
else {
    SendClientMessage(playerid, -1, "bye");
}