Change History (1)

comment:1 Changed 4 years ago by Дмитрий Лапшин (lapshin)

Resolution: задача сдана
Status: assignedclosed

Ух!

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

  1. Ну проверить аргументы оно предположим справилось. И то не все.
  2. А дальше оно то падает, то не падает. В любом случае валгринд и файл неправильный (и заголовком, и содержимым).

3.

Стиль:

  1. Работать с заголовком как набором байт это потеря и твоего, и моего времени.
  2. Магических констант дофига.
  3. Читать пиксель можно было бы и целиком за раз. А то и много за раз!
  4. Отступы!
  5. crop можно куда проще:


for (size_t i = 0; i < height; ++i)
    for (size_t j = 0; j < width; ++j)
         new_data[i][j] = old_data[y + i][x + j];
  1. То, что картинка хранится снизу вверх, проблема хранения, а не постоянной беготни с координатами.

5.

Note: See TracTickets for help on using tickets.