HW #1 (BMP) Kuznecz Anton hw_01

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

На всех тестах ругается валгринд. Даже на поворот лены 0 0 512 512

Течёте по памяти. При этом вывод правильный только вот этого теста, в остальных случаях файл получается неправильный.
Но, кажется, что ошибка не смертельная.
Проверьте, что вы правильный size в заголовке bmp сохраняете (обратите внимание, что этот size в двух заголовках встречается)
Причём кажется даже, что на файлах со строками, некратными четырём всё работает - проблема только в заголовке.

Стиль:

• int skip = (4 - (3 * width) % 4) % 4; - просится в отдельную функцию (с переименованными константами)
• кажется, что есть более оптимальный способ выделить память, чем 512 раз вызывать malloc
• хочется добавить всяких функций, которые помогут избежать дублирование кода
• проверте, что там по const в аргументах функций

А в целом это хорошая работа, молодец. Надо поправить всего ничего
Корректность: 4 (потому что тестов мало проходит)
Стиль: 6

[Anton]

1) Пофиксил valgrind
2) Добавил изменение size_image
3) Вынес skip в define
4) Память выделяется в два malloc
5) const в аргуметах функций

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

Тесты проходят
Стиль:
На самом деле всё хорошо, написано правильно, могу докопаться до двух мест:

• fread можно делать быстрее, если читать строку полностью
• менять size в заголовке файла, наверное, надо в уже crop, а не в rotate

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

Можете приступать к бонусу

[Anton]

Бонусное задание

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

Осталось единственное замечание к стилю

fread можно делать быстрее, если читать строку полностью

Корректность: 20
Стиль: 9
Бонус: 10