WW #4

[Sokolov Viacheslav]

Отличная работа, код легко воспринимается, комментарии по делу.

Есть проблема.
результат вызова malloc(0) не специфирован
​​https://en.cppreference.com/w/c/memory/malloc

If size is zero, the behavior is implementation defined (null pointer may be returned, or some non-null pointer may be returned that may not be used to access storage, but has to be passed to free).

В частности, если elements == 0, mergesort может вернуть -1.
Также не разрешены массивы размера 0:
​https://en.cppreference.com/w/c/language/array

If expression in an array declarator is an integer constant expression with a value greater than zero and the element type is a type with a known constant size (that is, elements are not VLA) (since C99), then the declarator declares an array of constant known size:

По этим причинам случай 0 элементов не работает.

Что можно сделать лучше:

• проверить, что не переполняется element_size * elements
• не терять константность: return strcmp(*((CONST char**) p1), *((CONST char**) p2));
• можно сделать один while и уменьшить копирование кода, если проверять условие

  while(!(i == right_bound1 && j == right_bound2))
  

[Solovyev Gleb]

Все недочеты исправил.