Исходное сообщение
"Выпуск Red Hat Enterprise Linux 6.9" Отправлено Аноним, 22-Мрт-17 15:50
> Дублирующиеся магические строчки: 329 и 337, 443 и 451, 850 883 и 908 > p = strappend("/run/systemd/sessions/", session); > p = strappend("/run/systemd/sessions/", session); Правильно. Нужно было создать единую функцию strappendRunSystemdSessions. Зато не "магически" и не "дублируется". Даешь также функцию strappendRunSystemdSeats и так далее. И после этого код, конечно же, станет читабельнее. > я честно не могу понять, почему, например, на 850 результат процедуры get_files_in_directory записывается в переменную, названную r > результат > r > r[esult] Ну, видимо ты из тех, кто не использует i для счетчика, l для количества и т. д. Есть определенные символы, которые имеют однозначный смысл. > Брезгует фигурными скобками Это код-стайл такой. Аргументик-то так себе. Чисто вкусовщина. Как я и говорил выше, "а у вас пустая строчка после ифа отсутствует". > На 238 пушка просто: /* We override the glibc assert() here. */. Молодец, Леня! А теперь попробуй потратить более 10 секунд и попытайся ответить, зачем он был переопределен. Слабо, очень слабо! Я ожидал что-нибудь типа option+15. А в ответ одни лишь "фигурная скобка стоит не там, пробел поставили не здесь, переменную назвали не так".