HW_01 kirill.kondratyuk

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

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

1. Один из заголовков о размере неправильный.

12/20.

Стиль:

1. struct bmp_image можно не паковать.
2. malloc не проверяется совсем.
3. Зачем выставлять функции, читающие отдельно заголовки, отдельно тело?
4. А если в bmp.c выделяется память, почему освобождается в main? Надо бы чётче границы отвественности провести.
5. static?
6. Волшебные числа просятся в константы.

5/10.

[kondratyuk.kirill]

Здравствуйте!
Нашел ошибку в заголовке(она ведь про размер файла?) и исправил её

Убрал пакование с bmp_image
Теперь все malloc проверяются
Функции чтения теперь объединены в одну(я просто думал сначала, что логично было бы разделить их)
Теперь память выделяется и освобождается функциями из bmp.c
static -и прикреплены
Из волшебных чисел остался только ноль

[kondratyuk.kirill]

Еще теперь выделение памяти идет в два маллока.

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

Но не починилось, теперь два неправильных( Стало 10.

Стиль:

1. Структуры можно присваивать и не по полям.
2. Может всё же main будет решать, как и какие писать сообщения?

8/10.

В сумме больше, но блин.

[kondratyuk.kirill]

Здравствуйте!
Вроде как теперь то уже все исправил!

С уважением, Кирилл

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

20!

Ну и зачем в заголовке объявлять static-функцию? А ceil_to_four не static. Да и ошибки выделения памяти всё ещё сами пищат.