WW #5

[Дмитрий Свиридкин]

По состоянию на 18 октября:

1. Лишний перевод строки при выводе точки (из-за него не проходят тесты)

2. Лишние файлы в репозитории

3. include/clist.h:22:41: note: expected ‘void (*)(intrusive_node *, void *)’ {aka ‘void (*)(struct intrusive_node *, void *)’} but argument is of type ‘void (*)(intrusive_node *, FILE *)’ {aka ‘void (*)(struct intrusive_node *, struct _IO_FILE *)’} void apply(intrusive_list *list, void (*op)(intrusive_node *node, void *data), void *data);

~

src/main.c:114:12: warning: passing argument 2 of ‘apply’ from incompatible pointer type [-Wincompatible-pointer-types]

apply(l, save_bin, fout);

4. Код с преобразованием чисел можно вынести в отдельные функции, чтобы избежать копипасты.

5. Код с чтением и запиисью файлов тоже лучше оформить отдельными функциями.

6. Отладочные комментарии нужно в конце убирать

Не забывайте в следующий раз оформлять тикет.

5/10

[Дмитрий Свиридкин]

убирать ключи компиляции из-за ворнингов не стоит. Лучши их исправить.

Число аргументов лучше проверить.
У неиспользуемого аргумента можно убрать имя или использовать (void)(arg)

fout = (FILE*)fout; --- здесь ничего не происходит (Есть подозрения, что делалось совместно -- проверяйте хотя бы друг друга, когда ерунду пишете!)

Почему функция, сохраняющая точки в файл, удаляет их из списка??!

7.2/10