HW#1 (BMP) Kiselev Igor hw_01

[kiselev.igor]

Ту проблему, о которой я вам писал в слаке я так и не смог устранить.

[Антон Филатов]

Все тесты падают с exit status != 0
Правильно отрабатывают только тесты с некорректными входными данными

Стиль:

• вы читаете все символы за один fread - каеф (это не замечание, это, наоборот, похвала)
• забыли при вырезании файла изменить size в заголовке
• кажется, что вместо функции place есть стандартная, которая делает именно это
• в main.c строка 28 просится разбить её на несколько строк. Ну и строки 29-30 тогда уж в ассерт стоит переделать
• проверьте, что вы не забываете ключевое слово const

Корректность: 4/20
Стиль: 5/10

• для этого unsigned long memmoryX = (3 * img->header.width + 3) & (-4); хочется либо макрос, либо функцию отдельную

[Антон Филатов]

hexdiff

[Антон Филатов]

Что-то всё ломается. Визуально Лена поворачивается, например на тесте 0 0 512 512, но в бинарном сравнении с эталонным поворотом очень много несовпадений. В приложении пример hexdiff-а откуда-то из середины файла. Причём тот факт, что различается что-то около 60 - это не закономерность. Там в файле есть совершенно разные двойные байты, которые отличаются (но всегда на 1)
На неквадратных (и не делящихся на 4 изображениях) иногда, но не всегда завершается с ненулевым кодом возврата
Так что корректность по-прежнему
Корректность: 4/20

Стиль:

• 256 - в константу
• хочется разбить load_bmp на более короткие функции, но это уже придирки

Стиль: 8/10

[Антон Филатов]

Вот сейчас практически всё работает, но

забыли при вырезании файла изменить size в заголовке

Везде вы пишете неправильные заголовки.
Вы прекрасно справляетеся с паддингами, но везде что-то чудите в паре байт в заголовках. (к тому же напоминаю, что размер файла указан сразу в двух заголовках)
14/20
Стиль: 10/10