Opened 5 years ago
Closed 5 years ago
#393 closed ожидается проверка (задача сдана)
WW #5
Reported by: | Filippov Denis | Owned by: | Дмитрий Свиридкин |
---|---|---|---|
Component: | WW_c_io | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 5 years ago by
Owner: | changed from Дмитрий Свиридкин to Filippov Denis |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 5 years ago by
Owner: | changed from Filippov Denis to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 1.0 → 2.0 |
comment:3 Changed 5 years ago by
Owner: | changed from Дмитрий Свиридкин to Filippov Denis |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Тесты пройдены
По одной из веток в main будет утечка памяти. main можно переписать так, чтоб код, выполняющий очистку памяти вызывался без ветвлений, один раз в самом конце.
Повторные вызовы (container_of(node, point, node)) лучше заменить одним с созданием переменной для элемента.
От артефактов тестирования в коде стоит избавиться, либо аккуратно оформить main набором функций, к которым можно написать тесты в отдельном файле (чтоб было unit-тестирование совсем по фен-шую)
10.5/12
comment:4 Changed 5 years ago by
Owner: | changed from Filippov Denis to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 2.0 → 3.0 |
comment:5 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Выходной файл не закрыли
11.5
Note: See
TracTickets for help on using
tickets.
Течет память. Порядок байтов перепутан.
Segfault на пустом списке
Объявляйте переменные как можно ближе к точке их использования: для этого есть стандарт C11.
Используйте типы с фиксированной длиной (int32_t, uint8_t) вместо int, char и прочих при работе с бинарными данными.
5/12