WW3

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

1. Ворнинги компиляторов мы не любим, от них нужно избавиться. К тому же все ворнинги в твоем случае по существу.
2. Строки не сортируются и портят память, на экране мусор, а valgrind ругается (а за возмущения валгринда -- уже автоматом идет 0/7)
3. Если функция что-то возвращает, то инструкция return <значение> должна выполняться при любых вариантах работы функции, иначе она может вернуть мусор. У тебя для mergesort это не выполняется.

Корректность -- 0/7.

Стиль -- 0/3:

1. Длинные строки (>80 символов) нужно разбивать на несколько. Примеры того, как строки должны разбиваться для условий, вызовов функций, объявлений функций и т.д. можешь посмотреть вот тут -- ​​https://google.github.io/styleguide/cppguide.html#Formatting
2. Не нужно жалеть пробелов вокруг знаков арифметических операций и пустых строк для разделения кода на смысловые блоки.
3. Вот так обычно не пишут:

   }
   else {
   

Лучше писать

} else {

4. Там, где ты копируешь память, лучше использовать синтаксис массивов, а не указателей.
5. В main.c во всех циклах for у тебя пробелы расставлены не по людски.
6. В main.c ты выделяешь динамический массив на стеке data[argc - 2], если входных данных слишком много -- стек переполнится.
7. Что это за конструкция? char* a = &data;

data -- уже имеет тип int* -- это указатель на начало массива. &data имеет тип int** и это указатель на указатель на начало массива. Из-за этого ты пишешь совершенно не в свою память и корраптишь стек.

8. При выводе данных не нужно городить макароны из арифметики указателей, пользуйся синтаксисом массивов.
9. В случае сортировки строк твой алгоритм, который сначала находит максимальную длину, а потом копирует строки в один здоровый буфер, во-первых неоптимален по памяти -- представь себе входные данные из одной строки длиной тысячу символов и тысячи строк длиной в 1 символ, а во-вторых совершенно зазря копирует строки целиком при сортировке, что совершенно не нужно. Ты можешь в массиве хранить не сами строки, а указатели на них и сортировать указатели, вместо самих строк.
10. Используешь необъявленную функцию strtol. Она объявлена в stdlib.h

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

Корректность 0/7
Валгринд по прежнему ругается на утечки при сортировке char'ов.

Стиль -- 2.8/3

1. Всё равно почему-то жалеешь пустых строк для разделения кода на логические блоки. Открой, к примеру, исходник какого-нибудь ​файла написанного на си и посмотри как код в нем разделен пустыми строками на блоки. Постарайся сделать в своем файле что-то похожее.
2. В заголовочном файле длинную строчку тоже стоит разбить.

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

10/10.

Бонус, к сожалению, ты не получаешь.
Потому что ты не проверяешь результат рекурсивных вызовов функции mergesort. Если в одном из них память выделить не удастся, то вызывающая функция об этом не узнает и продолжит работу.