WW_intrusive_list Mittcev lab_04

[mitcev.nikita]

Я не очень уверен, можно ли делать typedef с таким же именем, как и сам struct, вы про это как-то неоднозначно сказали на паре. Если что поправлю.

[Антон Филатов]

Я не очень уверен, можно ли делать typedef с таким же именем, как и сам struct, вы про это как-то неоднозначно сказали на паре. Если что поправлю.

нет строгого правила. Скомпилировалось - значит, можно (это моё мнение)

автотесты пройдены. И даже валгринд не ругнулся. Вы молодец. 8/8 за корректность

Стиль:

• а какой смысл вот этого intrusive_node* head = &(list->head);? чтобы в будущем писать head вместо list->head? это не ошибка, пишите, как вам удобнее, но для меня этот момент неожиданный
• вот здесь

typedef struct intrusive_list {
    intrusive_node head;
} intrusive_list;

вы уверены, что вы хотите голову на стеке выделить? То есть вы, судя по всему всё правильно с памятью сделали, раз она у вас не течёт, но вообще говоря, неожиданное решение

• а почему вы некоторые инклюды делаете в кавычках, а некоторые - в треугольных скобках?

(да, я уже до фигни докапываюсь, извините)

Стиль: 3/4

Итого: 11/12

[mitcev.nikita]

Исправил кавычки + переделал head на указатель. Голова выделялась на стеке только потому, что мне не нравятся malloc-и, может быть я просто слегка не привык к ним еще.

[Антон Филатов]

Принимается, 12/12