HW #1

[Vasily Alferov]

Круто.

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

8/20.

• Аргументы перепутаны: либо ты пропустила crop-rotate, либо забыла, что нулевой аргумент — название файла. Напоминаю интерфейс:

  $ ./hw_01 crop-rotate samples/lena_512.bmp out/lena_512.bmp 0 0 512 512
  

• Кое-где путаешь h и w, из-за этого падают тесты, где w != h. У меня перманентные сегфолты. Пример теста:

  $ $ ./hw_01 crop-rotate samples/lena.bmp lena.bmp 0 0 30 40
  Segmentation fault (core dumped)
  

• По модулю первых двух замечаний тесты без паддинга проходят.

• Паддинг не реализован, надо реализовать.

Без паддинга половина баллов, ещё минус балл за аргументы и минус балл за h и w.

Стиль

8/10.

• Неплохо бы сделать функцию imageMalloc статической и не выносить в хедер, так как она чисто внутренняя.
• Надо бы единый стиль в наименованиях равнозначных переменных: либо NotRotated и Rotated, либо notRotated и rotated. Второе, вообще-то, обычно препдочтительнее: не видел ещё, чтобы локальные переменные называли с большой буквы; глобальные кое-где всё же называют.

[Vasily Alferov]

Круто, зачтено.

Стиль: 10/10
Корректность: 18/20

• В save_bmp сначала записала значения заголовка в файл, а потом поменяла. Наверное, имелось ввиду наоборот.

• Путаешь x и y в crop и отсчитываешь не от того угла =(