WW #2

[Святослав Власов]

За корректность -- пока 0, т.к. требования к оформлению не соблюдены и нельзя запустить тестирующую систему.

1. Сигнатуры функций не соответствуют определенным в стандарте (ключевые слова, порядок аргументов)
2. Undefined behavior в функции test_strcat:

2.1 Никто не гарантирует, что память выделенная на куче будет заполнена нулями, там может быть что угодно.
2.2 Пишешь за границами памяти.

3. Память в тестах не очищается.

За стиль -- 1.5
Замечания:

1. Лишние заголовочные файлы (stdlib.h в str.c, stdio.h в test_str.c)
2. Неровные отступы
3. Мало автотестов. Нужно написать хотя бы по паре тестов на основные варианты работы функций. Например, для my_strcmp убедиться, что варианты с возвращением 0, 1 и -1 работают корректно, для функций my_strcat убедиться что корректно работают случаи с пустой строкой в качестве одного из аргументов.

[Terova Valeriia]

Не нашла ошибки в сигнатурах функций :(

Все остальное вроде бы пофиксила.

[Святослав Власов]

По прежнему сигнатуры не исправлены.
Открой стандарт ​http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1256.pdf найди там сигнатуры функций, прочитай что откуда копируется и что должно быть возвращено.
Вот пример для strcpy:
https://trac.compscicenter.ru/spbsu/attachment/ticket/50/photo_2020-11-22_02-57-21.jpg
Прочитай под пунктом 2 что куда должно копироваться.
Аналогично нужно сделать для остальных функций.

А пока 0/7 за корректность.

Плюс к этому my_strcpy пишет на один символ за границы переданного буфера.

За стиль претензий нет -- 3/3

[Святослав Власов]

Даю возможность исправить до вечера воскресенья.

[Terova Valeriia]

Исправила сигнатуру для my_strcpy, все остальное вроде бы ok (я честно до этого момента 4 раза читала этот стандарт и не замечала, что там что-то не так, извините :)).

За буфер теперь тоже писать не должен.

🙏🏼🤞🏼

[Святослав Власов]

6/7 корректность -- my_strcpy и my_strcat по прежнему пишут за буфер: сначала нулевой байт записывается в цикле while, а потом еще раз записывается отдельно.

Минус балл за позднюю сдачу.

Итого 8/10.