Показательные грабли

[guz]

Словил вчера две шикарные ошибки, советую больше никому такие не ловить:

1) Многострочный макрос:

#define HELLO(X) printf("Hello, "); \
                            printf("%s!\n", (X))


//тут много кода, пока его писал забыл что такое HELLO()
//и так, беззаботно
   if (name) HELLO(name);
//угадайте, что выводится?

2) Переопределние имени:

int var = 0;

do
{
     var = do_smth_1();
     if (var) break;
     var = do_smth_2();
     if (var) break;
     var = do_smth_3();
     if (var) break;
     var = do_smth_4();
     if (var) break;
     int var = do_smth_5();
     if (var) break;
     var = do_smth_6();
     if (var) break;
     var = do_smth_7();
     if (var) break;
     var = do_smth_8();
     if (var) break;
     var = do_smth_9();
     if (var) break;
     var = do_smth_10();
     if (var) break;
     var = do_smth_11();
}
while(0);

if (var)
  printf("ERROR\n");
else
  printf("SUCCESS\n");

Если var не ноль после do_smth_5 и далее получаем "SUCCESS".
Без всяких предупреждений компилятора.


Всем успехов, и не наступать на такие грабли.

[guz]

Вторая проблема решена опцией -Wshadow компилятора.