Opened 4 years ago
Closed 4 years ago
#505 closed ожидается проверка (задача сдана)
HW #1
| Reported by: | Tarabonda German | Owned by: | Дмитрий Свиридкин |
|---|---|---|---|
| Component: | HW #1 (BMP) | Version: | 3.0 |
| Keywords: | Cc: |
Description
Change History (4)
comment:1 Changed 4 years ago by
| Version: | 1.0 → 2.0 |
|---|
comment:2 Changed 4 years ago by
| Owner: | changed from Дмитрий Свиридкин to Tarabonda German |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
comment:3 Changed 4 years ago by
| Owner: | changed from Tarabonda German to Дмитрий Свиридкин |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 2.0 → 3.0 |
comment:4 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
В заголовках неправильные размеры прописаны и перепутана нумерация строк, а так работает.
WORD и DWORD, конечно, смотрятся здорово, но лучше оставить uint16_t и uint32_t. Так сразу понятны размеры.
В заголовках функций параметры лучше именовать. Что означают параметры crop (4 int) -- никому, кроме вас, неизвестно.
функции count_align и change_BMP_data в заголовке лишние. Они не являются частью интерфейса.
17 + 7
Note: See
TracTickets for help on using
tickets.
34й байт неправильно заполняется.
Падение на неквадратной области.
Код, расчитывающий выравнивание, дублируется. Также дублируется код, обновляющий заголовок.
Вместо констант 1,2,3 для ошибок лучше завести enum.
8 + 7