Opened 3 years ago
Closed 3 years ago
#255 closed ожидается проверка (задача сдана)
hw01 haha
Reported by: | petrov.vladimir | Owned by: | Святослав Власов |
---|---|---|---|
Component: | HW #1 (BMP) | Version: | 2.0 |
Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
Поправил предыдущие ошибки и добавил решение дополнительного задания
comment:3 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Задание 30/30
На бонусе валгринд детектит ошибки. Ты читаешь не из своей памяти. А читаешь потому, что забыл, что после того, как прочитал строчку fread надо поставить завершающий ноль. А еще лучше было читать используя fgets.
Note: See
TracTickets for help on using
tickets.
Корректность -- 20/20
Тесты все прошли с первого раза, молодец! Можешь делать бонус.
Стиль -- 8/10
printf("%s", "Cannot create new pixels\n");
, можно простоprintf("Cannot create new pixels\n");
, а еще прощеassert(bmp->pixel && "Cannot create new pixels");
fread(&bmp->header, sizeof(bmp_header), 1, file) == 0
-- лучше... != sizeof(bmp_header)
, если хедер прочитается не до конца, явно что-то пошло не так.memcpy
кажется что получше смотрится:crop
ты можешь копировать куски строк не попиксельно, а целиком, все размеры у тебя для этого есть.