WW_mergesort lab_03 Kozyrev Andrei

[Антон Филатов]

автотесты отработали правильно с точностью до лишних пробелов в конце строки
7/7 за корректность
Стиль:

• в файле mergesort.h должна быть объявлена только одна функция. Если вы хотите пользоваться дополнительными функциями, их следует объявлять в mergesort.c
• Вот такие вещи

  // Create an array of appropriate type
  // Things like the one below almost made me crazy
  for (int i = 0; i < argc - 2; i++)
      *(array + i) = **(argv + 2 + i);
  

  можно делать так:

for (int i = 0; i < argc - 2; i++)
   array[i] = argv[i + 2][0];

• постарайтесь в файлах, которые вы в итоге заливаете, убирать комментарии. Понятно, что они нужны вам, храните тогда две версии файла - одна для вас, одна для меня

В остальном норм. Немного перегружены дублированием условия на 30й и 37й строчках. Кажется, что можно эту ситуацию разобрать изящнее, но да ладно, не буду докапываться. Замените везде *(array + i) на array[i], пожалуйста.

I don't get why do we need to assure ourself, that we allocated some memory
Do not understand what can go wrong
Ну, например, вы запускаете вашу программу на чайнике, который имеет 4 КБ оперативной памяти, из которой свободно 9 байт и пытаетесь аллоцировать 10 байт - тогда malloc вернёт вам 0

Стиль: 1,5/3
Итого 8,5/10

[Andrei Kozyrev]

Изменения внесены
Поправил все, что вы просили

[Антон Филатов]

Принимается 10/10
Любопытно, что вы сделали int mergesort с заделом на бонусное задание, но так его и не реализовали.