WW #2

[romanov.daniil]

my_strcpy и my_strcat падают по assert. Разобраться с этим делом, к сожалению, не успел.

[Vasily Alferov]

• Вообще, не компилируется.

  $ make
  mkdir obj
  gcc -Wall -Wextra -Werror -Iinclude -c src/test.c -o obj/test.o
  gcc -Wall -Wextra -Werror -Iinclude -c src/test_str.c -o obj/test_str.o
  In file included from src/test_str.c:3:
  src/test_str.c: In function ‘test_my_strcat’:
  src/test_str.c:16:28: error: comparison with string literal results in unspecified behavior [-Werror=address]
       assert(my_strcat(c, b) == "abcdefg");
                              ^~
  src/test_str.c:16:28: error: comparison with string literal results in unspecified behavior [-Werror=address]
       assert(my_strcat(c, b) == "abcdefg");
                              ^~
  cc1: all warnings being treated as errors
  make: *** [Makefile:15: obj/test_str.o] Error 1
  

Всё потому, что строки нужно сравнивать через strcmp. Если ты сравниваешь их на ==, ты сравниваешь указатели.

• Ещё у тебя маловато зависимостей в Makefile, после изменения кода файлы не пересобираются.

• Ну и strcat и strcpy должны возвращать старое значение s1, а не новое.

Других замечаний у меня нет ни по корректности, ни по стилю. Давай договоримся, что тут пока 0, а ты исправишь то, что не компилируется/не работает, и станет 10?

[romanov.daniil]

Все замечания учтены и исправлены.

[Vasily Alferov]

Зачтено.