Исходное сообщение
"Выпуск Red Hat Enterprise Linux 6.9" Отправлено Аноним, 22-Мрт-17 15:30
Первый попавшийся файл: https://github.com/systemd/systemd/blob/master/src/libsystem... Дублирующиеся магические строчки: 329 и 337, 443 и 451, 850 883 и 908. Хардкода столько, что я даже не буду начинать перечислять. Практически все переменные - однобуквенные и совершенно непонятные (я честно не могу понять, почему, например, на 850 результат процедуры get_files_in_directory записывается в переменную, названную r). Брезгует фигурными скобками, никакой последовательности в стиле. Особенно феерично смотрятся строки 832 и 833. Странное разделение всех объявленных процедур на _public_ и static - опять же, бардак в стиле (ну обозвал бы _private_, раз на родное похапе так тянет, хоть бы выглядело однородно). Там еще подключается файл macro.h - вот там тоже много веселья, у меня даже глаза разбегаются: https://github.com/systemd/systemd/blob/master/src/basic/mac... На 238 пушка просто: /* We override the glibc assert() here. */. Молодец, Леня! В макросе на 323 ошибка: не учитывается знак минуса для отрицательных чисел (а они, судя по комментарию для соседнего макроса на стр. 313-316, ожидаются). Опасный макрос на 333. Какой-то ад на строках 335-361. Опять же, непоследовательный стиль: где есть вертикальное выравнивание для \ в макросах, а где нет. Про однобуквенные переменные и абсолютно нечитабельные битовые хаки и начинать уже не хочется. Уверен, что есть и еще. Плохой демагог. Очень плохой демагог!