Change History (1)

comment:1 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Resolution: задача сдана
Status: assignedclosed

Корректность:

  1. Громко воет Valgrind. Он ругается, что add_node пишет куда нельзя. Почему?
  2. struct point *p = malloc(sizeof(p)); Эхх… sizeof(p) == sizeof(void *).
  3. remove_point: use after free. Ты сначала освобождаешь память, потом обращаешься туда, чтобы сделать next.
  4. Твой offsetof undefined behaviour. Используй готовый из stddef.h.
  5. scanf может выйти за пределы буфера.

Стиль:

  1. void init_list(struct intrusive_list *intrusive_node) {

    Если на клетке слона прочтёшь надпись «буйвол», не верь глазам своим.

    Имя аргумента можно было аккуратнее. Скажем, list.

7/12.

Note: See TracTickets for help on using tickets.