HW #1

[yakushina.tatyana]

Извините, пожалуйста за задержку. Отпарвила только save и load. Доделаю crop и rotate. Спасибо за понимание.

[Артур Гулецкий (huletski)]

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

• память выделенная в main.c:9 не освобождается;
• отсутствуют проверки числа и корректности аргументов, переденных программе;
• выходное изображение записывается не полностью (размер файла с прямоугольным изображением не совпадает с ожидаемым, хотя визуально файлы очень похожи);
• проблема с portability при учете padding’ов: стандарт не гарантирует определенное бинарное представление -4 в дополнительном коде.

Стиль

include/bmp.c

12: типы полей должны быть фиксированного размера, как в описании формата bmp;
25: если используете структуру для чтения/записи пикселей в файл, нужно было отключить выравнивание;
25: последовательность цветовых каналов в bmp BGR;

include/main.c

2: избыточное подключение stdio.h;
9: отсутствует проверка значения, которое вернул malloc;

src/bmp.c

10, 20, 60, 61, 62: отсутствует проверка корректности выполнения fread, malloc, fopen, fwrite, fseek;
11, 22: дублирование логики учета padding’ов, нужно вынести в отдельную функцию;
30: индексы лучше объявлять типа size_t;
32: перед продолжением statementa не хватает дополнительных отступов.

Баллы: корректность 14, стиль 6.