hw_01 prudnikov.mark

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

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

1. Valgrind: память надо не терять. А ещё иногда промахиваешься при чтении...
2. Корректность: как минимум не в ту сторону крутишь. И кажется не вырезаешь.

Я пока не могу зачесть.

Стиль:
1.

#pragma pack(push, 1)

#include <stdlib.h>
#include <stdio.h>
#include <string.h>

Стандартные структуры совсем не хотят этого. Вообще эти прагмы надо стягивать на самое маленькое пространство.

1. Проверяй malloc!
2. Пробелы.
3. Не знаю до конца зачем bmp_t ты хранишь на куче.

Но так выглядит нормально, откровенных проблем в crop не вижу.

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

У тебя -Werror: сравнение беззнакового с нулём.

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

А если это поправить то работает, так что давай ты шустро пробежишь глазами, где signed где unsigned)

[Марк Прудников]

Извините, не понимаю. У меня в main написано: uint32_t x,y,w,h;, и это закоммиченно, проверил на Subversion. У меня -Werror не вылетает, когда make делаю. Может тогда я проще откажусь от идеи с использованием stdint?

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

Так как раз в этом и беда, uint32_t x, x < 0 быть не может. Может int32_t?

С ними работает идеально, поэтому я зачёл и допуск дал, за это не переживай.

Корректность: если поправить глупость то шикарно, 15.

Стиль:

1. Пробелы!

9/10.

[Марк Прудников]

Поправил типы.

Лишние пробелы, где нашёл, убрал.

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

Стало СИЛЬНО хуже!

Вообще хватило бы использовать int32_t для координат, а ты по дороге наломал много чего.