WW_#4

[Артур Гулецкий (huletski)]

Решение не собирается

{lab_04}[2319]$ pwd && svn up && svn status
/home/hfx/dvl/cpp19/kirillova.evgeniya/lab_04
Updating '.':
At revision 1327.
{lab_04}[2319]$ make
mkdir obj
gcc -c src/main.c -o obj/main.o -Wall -Werror -Wextra -I./include
cc1: fatal error: src/main.c: No such file or directory
compilation terminated.
Makefile:8: recipe for target 'obj/main.o' failed
make: *** [obj/main.o] Error 1

[kirillova.evgeniya]

Простите, пожалуйста. Я знаю, что пропустила все дедлайны, просто я очень сильно тупила именно с этой лабой. Простите еще раз

[Артур Гулецкий (huletski)]

Тесты проходят -> +10.

Замечания:

• не соблюдается единый стиль отступов (e.g. mergesort.c:20) -> -1;
• mergesort.c:63. Копировать массив можно было целиком (а не поэлементно) при помощи my_memcpy;
• в коде есть строки по 100+ символов. В общем случае их надо избегать, так как код становится менее читаем. Т.к. речь идет о сигнатурах, то решайте сами исправлять или нет;
• main.c:28. Лучше arr, а не mas.

Итог: 9, чините отступы.

[Артур Гулецкий (huletski)]

Ok, +10.

Замечания:

• если сигнатура занимает две и более строки из-за переносов, перед второй и последующими обычно ставят отступы/выравнивают, ориентируясь на параметры из первой строки;
• mergesort.c:63. Можно было вызвать my_memcpy для всего массива сразу, а не поэлементно;
• mergesort.c:7. Лучше назвать параметры dst (destination) и src (source).