That's better, but you've posted the code without any indentation; eg,

void msDelay(tInt msec) 
{ 
tInt cntr_1, cntr_2; 
for (cntr_1 = 0; cntr_1 <= msec; ++cntr_1 ) 
{ 
for ( cntr_2 = 0; cntr_2 <= 5000; ++cntr_2); 
} 
}

would normally be laid out something like this:

void msDelay(tInt msec) 
{ 
   tInt cntr_1, cntr_2; 
   for (cntr_1 = 0; cntr_1 <= msec; ++cntr_1 ) 
   { 
      for ( cntr_2 = 0; cntr_2 <= 5000; ++cntr_2); 
   } 
}

where the indentation indicates which lines are "inside" the blocks formed buy other lines.

Have you verified that this delay routine does actually give the correct delay?