WW #4 chistyakova.polina

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Динамические массивы на стеке это ай.
2. calloc проверить бы.
   1. Кстати, почему ты так любишь calloc? malloc?

Стиль:

1. UNTITLED22_MERGESORT_H лол.
2. Дохлый код в комментах :/
3. Перенос нормальный план, но можно аккуратнее:

   void merge(
       void *array,
       size_t left, size_t mid, size_t right,
       size_t element_size,
       int (*comparator)(const void *a, const void *b)
   ) {
   

7/10, очень классно.

[chistyakova.polina]

Корректность:

1. Динамические массивы исправлены
2. calloc мне нравится больше... Испытываю неосознанный страх перед неопределёнными вещами (UB совсем какие-то странные бывают, максимально защищаемся от них)

Стиль:

1. А что не так-то? *всё равно исправила*
2. Это был не дохлый код... Это были тесты... Чтобы если кто-то что-то изменил, он сразу раскомментил и проверил
3. Вот про перенос вообще не поняла. Зачем? Это же максимально портит вид(( Как будто я любым способом пытаюсь увеличить кол-во строчек...

[chistyakova.polina]

Перенос добавила красивый...

[Дмитрий Лапшин (lapshin)]

Корректность:

1. Valgrind нашёл лажку, память течёт.

Но всё красиво) 9/10, дочини.

[chistyakova.polina]

Я всё исправила

[Дмитрий Лапшин (lapshin)]

Молодец!