HW #1 potryasaeva.anna

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

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

1. Пожалуйста, не отправляйте решения, которые включают санитайзер. Valgrind не ценит.
2. Чот он ни одну картинку не справился обработать. На каких-то тестах содержимое картинки неверно, на каких-то падает (с valgrind).
3. Аргументы проверять?

0.

Стиль:

1. Кстати, CFLAGS в Makefile бы тебе помог.
2. assert("Unknown arguments") делает не то, что хочется. Надо что-то приличнее.
3. В структурах: int16_t это хорошо, но почему int?!
4. Зачем выделяешь BMP на куче?

7.

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

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

1. Недопроверила аргументы (-1 -1 -1 -1). А местами перепроверила (зарубила хорошие).

Кроме глупой ошибки всё ок, 15.

Стиль:

1. Надеяться на размер int плохо. Используй типы известного размера (int16_t же знаешь!).
2. Есть некоторые вопросы к повторяющимся магическим числам.
3. Хранить изображение как char * вместо двумерной сущности затруднительно. Рекомендую двумерный массив структуры из 3 char.

7.

Полируй, в целом очень хорошо!

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

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

1. Ошибку не исправила:

   assert(x >= 0 && x + w <= bmp->header.biWidth && y >= 0 && y + h <= bmp->header.biHeight);
   

   против твоего:

   assert(x >= 0 && x + w <= bmp->header.biWidth && y >= 0 && y + w <= bmp->header.biHeight);
   

Стиль:

1. Какие-то вычисления вокруг магических констант всё ещё есть.

9.