HW #1 (BMP)

[Vasily Alferov]

Очень круто, мне нравится.

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

10/20.

• Базовый тест проходит, и это круто.
• Паддинг пока не реализован — надо реализовать. С учётом того, что всё остальное работает, это довольно просто — надо только честно реализовать.

Без паддинга половина баллов.

Стиль

8/10, пара небольших замечаний.

• В хедере у тебя include guard почему-то называется CLIST_H_, хотя хедер по-другому называется.
• В rotate: у тебя buffer используется только для того, чтобы хранить там указатели, а заголовки не используется. Мне кажется, логично тогда просто создавать указатель прямо внутри rotate и не требовать от пользователя функции буффер.

Вообще, даже если бы тебе нужны были и заголовки тоже, всё равно буффер логично было бы внутри создавать, а не извне брать. Буффер извне передают иногда, если структура тяэёлая или ресурсоёмкая и хочется дать пользователю функции возможность переиспользовать какие-то экземпляры, которые у него есть.

[Vasily Alferov]

Круто, зачтено.

По стилю больше замечаний нет.
Корректность: 18/20 =(

Какая-то беда в заголовках bfSize и biSizeImage =(
Вроде как они должны быть с учётом паддинга и тут я тебя обманул, сказав, что менять надо только load и save.

А ещё, кажется, область вырезается не та (надо было от левого верхнего угла отсчитывать).

По исправлению двух неточностей работает.

Это всё мелочи. Поскольку исправлять больше нельзя, тут снято совсем мало баллов.