На работе весело.
На работе код-ревью.
Приходят коллеги и рассказывают, почему мой код гавно.
А дальше начинается веселье.
Берём готовый кусок кода, который работает.
Правим косяки, найденные на ревью.
Вуаля. Код не работает.
На этот раз веселье получилось аж на полдня поисков.
Делаю сборку на своей тачке - всё ок.
Делаю сборку на сервере - не ок.
В процессе ковыряния оказывается, что мой кусок кода был просто... выкинут при сборке.
А всё почему?
Коллега 1 сказал на ревью, что этот код про Windows, и я завернул его в
#if defined(OS_WIN)
Коллега 2 сказал, что это наш код, и надо бы вынести его в отдельный модуль.
Сервер сказал, что сборка релизная, без всяких ускорений сборки (то есть, собираем по одному файлу за раз).
В итоге при сборке моего отдельного модуля в нём просто не оказалось OS_WIN.
Потому что не было правильного хидера. Который был на локальной машине, потому что там включен jumbo_build, который сливает все модули в одну кучу, и только потом их компилирует.
Нашёл. Поправил. Заработало.
Зато я теперь знаю, что WIN32 - это дефайн в SDK, _WIN32 - в компиляторе, а OS_WIN собирается из них в кастомном хидере, который ещё надо не забыть подключить. Ыых.
К чему я это. Мне таки кажется, что код-ревью - это скорее плохо, чем хорошо.
Даже не взирая на то, что в процессе можно найти косяки.
Ну или просто надо его проводить как-то по другому ;)
Ах да... Крайним оказался опять я. Наверное.
