WW #3

[kutyavin.denis]

Функции написаны, но в мейне пока нет консольного приложения.

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

Валгринд возмущается -- это сразу 0/7 за корректность.

Стиль -- 0/3

1. Используй синтаксис массивов там, где это возможно. В конструкциях вроде таких:

   *(dst + i) = *(src + i);
   

2. Зачем разбивать это на две строчки?

   	char* work_array = NULL;
   	work_array =  (char*) malloc((left_size + right_size) * element_size);
   

3. Длинные строки (>80 символов) нужно разбить.
4. Это условие бессмысленно, можно от него избавиться:

   if(right == right_size){
   

У тебя после завершения цикла while (left < left_size && right < right_size) гарантированно либо left == left_size, либо right == right_size, поэтому выполнится только один из циклов: while(left != left_size) либо while(right != right_size). Дополнительное условие здесь не нужно.

5. Эмм.. Как ты себе представляешь должна выполниться строчка с free?

   	return 0;
   	free(work_array);
   
   

6. В рекурсивных вызовах mergesort не проверяется их возвращаемое значение. Если в них что-то пойдет не так, вызывающая функция об этом не узнает и продолжит работу, что чревато утечками.
7. Снова, зачем разделять на две строчки?

   	int ans = strcmp(*x, *y);
   	return ans;
   

8. Не нужно писать свою реализацию перевода строки в число. Есть библиотечная atoi. То же самое касается вывода строки на экран. Достаточно написать printf("%s", str), а не выводить посимвольно.
9. Вместо выделения памяти под массив строк, можно просто передать в функцию argv.

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

Корректность -- 7/7
Стиль -- 3/3
Бонус -- 0.5/1

1. Не нужно вызывать второй рекурсивный вызов если первый завершился неуспешно. Вероятнее всего он тоже упадет, а если не упадет, то проделает бессмысленную работу.

10.5/11