WW #1

[Vasily Alferov]

В целом всё хорошо. Немного замечаний.

I. correct

Замечания по корректности.

1. make bin/io.o не работает после make clean, так как в таком случае папки bin не существует. Нужно сделать цель bin order-only зависимостью таргета bin/io.o и других объектных файлов. См. ​описание задания.

2. make clean после make clean возвращает ошибку, так как не может удалить несуществующие файлы. Чтобы он не ругался, нужно добавить флаг -f команде rm.

Замечания по стилю.

1. В C-файлах вида src/io.c обычно подключают include/io.h в первой строчке. И так для всех файлов. Почему — вам раскажут на лекции; если успею, повторю на практике. Ну и если тебе всё равно исправлять, исправишь ещё и это?

2. Более традиционное название переменной для флагов для языка C — CFLAGS, а не CCFLAGS. Я погуглил, CCFLAGS тоже встречается. Можно не исправлять, просто к сведению.

3. Давайте отделять хотя бы отдельные функции пустыми строками (в частности, от инклюдов)? Я абсолютно уверен, что это рекомендуется всеми популярными стайлгайдами для C.

4. В разных файлах отступы с разным числом пробелов (2 или 4). Не надо так. В рамках одной домашки желательно использовать один и тот же стиль.

5. Тоже небольшое замечание по стилю. Тернарный оператор в функции max: с пробелами творится что-то странное. Лучше окружить пробелам и ?, и :.

II. multiple

Не та ошибка, которую хотелось. Хотелось, чтобы это была ошибка линковки, то есть появлялась на сборке lab1, а не объектных файлов.

Замечания из I пункта сюда тоже относятся.

III. unresolved

Тут всё ок, но замечания из I пункта сюда тоже относятся.

[Vasily Alferov]

Для истории: я считаю, что пока оценка, которую я могу поставить за эту лабу, примерно 0.4.
Если тут появились исправления, стоит изменить этот тикет.

[Vasily Alferov]

Зачтено.