#define something() do { P1OUT = 123; P2OUT = 234;} while(0) 

Запишите без do {} while(0) чтобы слкдующая функция компилилась без ошибок:

void Test()
{
  uint8_t i;

  something();

  for( i = 0; i < 5; ++i)
     something();

  if(P1IN & 1)
    something();
  else
    printf("test");

}