HW #1 (BMP), Tulchinskiy, hw_01

[tulchinskij.aleksandr]

У меня возник вопрос по условию:
В BMP нумерация идет с левого нижнего угла (0,0 = левый нижний угол). Но при вырезании мы передаем координаты в нумерации с левого верхнего угла (0,0 = левый верхний угол). Это так и задумывалось, или условие на гите с ошибкой?

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

Вы сейчас делаете правильно - с верхнего левого угла. А в bmp пиксели лежат с левого нижнего, никакой ошибки нет.
Единственное, что вы делаете неправильно - вы перепутали местами x и y
В тесте
crop-rotate lena_512.bmp out.file 100 50 128 128
нужно отойти на 100 вправо и на 50 вниз,а вы делаете наоборот

И ещё одна мысль: вы правите в заголовке image size только в одном месте, хотя оно в каждом заголовке встречается по одному разу, то есть присутствует в двух местах.

Поэтому в целом все правильно, но надо поправить
Но пока по формальным критериям это 5/20 (за прохождение базовых тестов)
Стиль:

• проверьте, что вы нигде не забыли const

Бонус пока проверять не могу
Корректность: 5/20
Стиль: 9/10

[tulchinskij.aleksandr]

Поправил все.
По поводу доп задания - у меня был написан код и на прошлой проверке, но вы его не чекали :(. Поэтому я полез в тикеты людей с других групп и заметил одну деталь которой не было в гите: После вывода декодированного сообщения нужно поставить перевод строки. Надеюсь, что у вас такие же тесты и такой же чекер.

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

Сейчас все тесты проходят.
Но вот по бонусу беда. при кодировке вылезает очень много ошибок валгринда

Мой косяк. Всё нормально по валгринлу.

при декодировке закодированного вами файла на выходе получается пустая строка.
Пример файла кодировки - текстовый файл
0 0 R
1 2 G
3 2 B
и т.д.
Файл с сообщением - обычный текстовый файл

При декодировке ломается последний символ (была ".", стал какой-то символ из середины слова)
20/20 за корректность
10/10 за стиль
5/10 за бонус

[tulchinskij.aleksandr]

Теперь символы не по формату (не заглавные английские и не "," "." " ") пропускаются.

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

20 + 10 + 10