Opened 3 years ago

Closed 3 years ago

#198 closed ожидается проверка (задача сдана)

WW_io

Reported by: em.vladislv Owned by: Святослав Власов
Component: WW_c_io Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверка → ожидаются исправления

Корректность -- 2/7

  1. Бинарный формат не реализован

Стиль -- 1.5/3

  1. Какой сакральный смысл в этой строчке? void (*tmp)(struct intrusive_node *, void *) = op;. Чем просто op(current, data) не угодил?
  2. Если уж ты хочешь развернуть список, то не нужно функцию для этого называть insert. А во-вторых можно развернуть проще, просто добавив все ноды по очереди в другой список.
  3. Не надо падать в UB, если у тебя вдруг аргументов не хватает

comment:2 Changed 3 years ago by em.vladislv

Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

comment:3 Changed 3 years ago by Святослав Власов

Type: ожидается проверка → ожидаются исправления

Валгринд детектит утечки -- 0/7

Стиль -- 2.5/3

  1. Давай загрузку точек вынесем в отдельные функции.

comment:4 Changed 3 years ago by em.vladislv

Type: ожидаются исправления → ожидается проверка
Version: 2.0 → 3.0

comment:5 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: assigned → closed

10/10

Note: See TracTickets for help on using tickets.