HW #1 (BMP) sysoev.sergej

[Дмитрий Лапшин (lapshin)]

Корректность: 20.

Бонус: чот ему совсем плохо и краш. Даже понятно почему, на самом деле)

Стиль:

1. Что за swap макросами.
2. Вообще в этом задании можно было не играть с LE.
3. В чтении картинки вижу читаешь по пикселю. А как насчёт по строчке?
4. Твой crop забивает освобождать память. Это первый раз на моей памяти, когда такое предпринято) Не проблема, но если захочешь попробуй честно.
5. Перебирающие в stego цепочки if просятся на switch.

7/10.

Но для первой посылки это блеск.

[Sergey Sysoev]

Бонус: в задании нигде не было указано про перенос в конце строки, догадался только почитав чужие тикеты, возможно стоит добавить в задание... При выводе в терминал ещё можно догадаться, но вывод в файл можно и без \n делать.

По стилю:

• 1, 2, 5 исправил
• 3 оказалось нужно не только там исправить, писал картинку в файл и делал crop я тоже зачем-то по пикселю.
• 4 я дописал комментариями к существующему коду как аналог
• подсмотрел в чужие тикеты и вынес round_up_to_multiple_of_4 в отдельную функцию, заодно часть комментариев пропала.

[Дмитрий Лапшин (lapshin)]

20/10/10.