#464 closed ожидаются исправления (задача сдана)
HW #1
| Reported by: | Maria Chizhova | Owned by: | Maria Chizhova |
|---|---|---|---|
| Component: | HW #1 (BMP) | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 4 years ago by
| Owner: | changed from Дмитрий Свиридкин to Maria Chizhova |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
| Owner: | changed from Maria Chizhova to Дмитрий Свиридкин |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 1.0 → 2.0 |
comment:3 Changed 4 years ago by
| Owner: | changed from Дмитрий Свиридкин to Maria Chizhova |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
hw_01: src/bmp.c:37: crop: Assertion `x >= 0 && x + w <= bmp->header.biWidth && y >= 0 && y + w <= bmp->header.biHeight' failed.
При попытке повернуть целиком неквадратную картинку. Заголовок где-то неправильно обновился.
8 + 8
comment:4 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
comment:5 Changed 4 years ago by
if (bmp->image) free(bmp->image); free(bmp);
Для создания и удаления сложных объектов лучше всегда делать отдельную функцию, чтобы было проще поддерживать объект в консистентном состоянии.
20 + 8
Не забывайте обновлять тикет.
Note: See
TracTickets for help on using
tickets.
Что-то не так с последним столбцом пикселей. Проверьте на квадратной картинке, повернув целиком ее всю.
pragma pack достаточно навесить только на структуру для заголовка.
магическую константу 54 стоит заменить на sizeof(BMPHeader)
код с формированием заголовка дублируется -- лучше вынести в отдельную функцию.