Is this gsm interfacing program correct

#include<reg51.h>
#include<string.h>
#include<absacc.h>
#include<stdlib.h>
#include<ctype.h>

void delay(unsigned int i);
void tx0(unsigned char);
void SMSString(unsigned char*text) ;
void init();
void send_a_message();
//void call();


// Moblie no.of Owner
unsigned char code Mob_no[]= "+917792849639"; 




void delay(unsigned int i)
{
        unsigned int k, l;
        for(k=0;k<i;k++)
        for(l=0;l<1000;l++);
}

void init()
{
	P0=0x00;
	TL1=0XFD; //9600 @ 11.0592
	TH1=0xFD;
	TMOD=0x20;
	SCON=0x50;
	TR1=1;
}


void SMSString(unsigned char* text) //function to send SMS using GSM modem
{
	while (*text)
	{
		tx0(*text++);
	}
} 
       
void tx0(unsigned char x) //send data to serial port 0
{
	EA=0;
	SBUF=x;
	while(TI==0);
	TI=0;
	EA=1;
}

void send_a_message()
{
 SMSString( "AT+CMGF=1\r"); // select text mode for sms
 delay(50);

 SMSString("AT+CMGS=\"");
 SMSString(Mob_no);
 SMSString("\"\r");
 delay(1);
 SMSString("somebody entered incorrect password");
 delay(100);
 tx0(0x1A); // Ctrl+z
 delay(10);
}

/*void call()
{
 SMSString("ATD+917792849639;\r");
 
 delay(1);
 delay(10);
}
*/

void main ()
{

init();

SMSString("AT\r"); // AT commands to initialize gsm modem
delay(50);

SMSString( "ATe0\r"); // turn off echo
delay(50);

SMSString( "AT&W\r"); // save settings
delay(50);

send_a_message();
delay(200);

//call();
while(1);

}

void send_a_message()
{
 SMSString( "AT+CMGF=1\r"); // select text mode for sms
 [COLOR="#FF0000"]delay(200);[/COLOR]

 SMSString("AT+CMGS=\"");
 SMSString(Mob_no);
 SMSString("\"\r");
 [COLOR="#FF0000"]delay(200);[/COLOR]
 SMSString("somebody entered incorrect password");
 [COLOR="#FF0000"]delay(200);[/COLOR]
 tx0(0x1A); // Ctrl+z
 [COLOR="#FF0000"]delay(200);[/COLOR]
}