HW #1 (BMP) hw_01 rachkin.vladimir

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Один из заголовков про размер изображения неверен на всех тестах.

12/10.

Стиль:

1. implicit declaration.
2. Пробелы! Уже можно научиться писать return_type name(arg_type arg_name) {.
3. Вообще пустых строк многовато, особенно когда можно писать в одну вместо трёх: FILE *file = fopen(…);
4. Не проверяешь результат calloc/malloc.
5. Было бы хорошо, если bmp.c не решал сам, куда печатать сообщения об ошибках.

5/10.

[Дмитрий Лапшин (lapshin)]

Корретктность:

1. А теперь Valgrind на каждом тесте. 15/10.

Бонусу как-то фигово, там Valgrind и что-то странное вместо перевода строки в конце.

Стиль

1. Теперь выглядит солидно) Но волшебных чисел многовато в бонусе.
2. FILE *log такое внезапное решение, но и такое в целом нормальное, что лайк. Я бы просто возвращал коды, чтобы программа могла решить, как сообщать, ну ладно.

9/10.

[Владимир Рачкин (rachkin.vladimir)]

Вы не проверили бонусное задание

[Дмитрий Лапшин (lapshin)]

Вообще-то проверил, и ему фиговасто было

Кор: 20. Бонусу фиговасто: он почему-то две точки вывел, и валгринд орёт. 20 повлияло на оценку, бонус нет.