WW_mergesort

[Наталья Марцинковская]

Пока только int, не могу разобраться с тем, как считывать char и str - ругается компилятор

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

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

1. Не сортируются символы и строки
2. На пустом массиве лезешь не в свою память
3. Выделяешь память под инты неверного размера. sizeof(argv[2]) -- это sizeof(char*), а не sizeof(int), они могут отличаться.

Стиль -- 2.3/3

1. Съехали отступы
2. Не пиши вот так:

}
else{

лучше так:

} else {

3. Не проверяешь выделенную память ассертами

[Наталья Марцинковская]

Добавила строки и символы, сделала еще бонус, исправила стиль.

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

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

Стиль -- 2/3

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

res = a;
...
free(res);

когда можно написать

free(a);

4. Синтаксис массивов?

for(int i = 0; i < elements * element_size; i++){
	*(start + i) = *(res + i);
}

5. Кое где намешала пробелы с табуляцией -- у меня съехали отступы
6. Какой смысл копировать аргументы из argv в отдельный массив при сортировки строк?

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

[Наталья Марцинковская]

Постаралась исправить все красиво

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

10.9/11

Всё равно отступы разъехались.