HW #1

[Азат Валеев]

Реализовал промежуточную версию (вроде с Леной (0 0 512 512) работает, да и с неквадратным изображением тоже может, только стороны должны делиться на 4 по какой-то из координат) и какую-то проверку аргументов, получилось очень топорно, не знаю, как оформить это читабельнее.

[Святослав Власов]

Базовый тест работает (пока только он)
Корректность 5/20

Стиль -- 6/10

1. Я тебе советую создать структуру, где будут вместе лежать хедера и пиксели изображения, так как они непосредственно связаны друг с другом и лучше их держать вместе
2. В load_bmp лучше передавать указатель на память, куда функция запишет хедера, чем возвращать память оттуда. Это позволит тебе, к примеру, держать хедера на стеке. И лучше бы назвать её тогда load_headers.
3. Зачем ты читаешь и пишешь попиксельно, если ты можешь прочитать/записать строчку целиком за раз?
4. Зачем ты передаешь в функции тройной указатель? Единственное, что ты с ним делаешь потом -- это разыменовываешь. Может тогда проще двойной указатель передать?

[Азат Валеев]

Сейчас это точно должно работать, увы, без бонуса :(

[Святослав Власов]

По прежнему работает только базовый тест, а в остальных неправильно записывается image_size в хедерах выходных изображений.
15/20 за корректность

Стиль - 9/10

1. Выделение памяти под bmp_file прямо просится в отдельную функцию, чтобы в main не было копипасты.

В остальном код хороший и лаконичный, хвалю.

[Азат Валеев]

Да, увидел, сейчас точно хэдера должны быть корректными :)
Написал функцию выделения памяти под пиксели, код в main стал компактнее.

[Святослав Власов]

Окей, только надо поместить его было в bmp.[h/c], а не в main.c
30/30