3 votes

another Merge Sort

Three things pop out to me: This is not a stable sort. Probably does not matter, as you currently only support integers. This only support integer sorts. If you look at the standard qsort you will ...
Martin York's user avatar
2 votes

Implementing an improved merge sort that uses insertion sort, with "levels" - Would this work correctly?

This question is marginally on-topic since you do not understand the behavior of the code that you wrote and maintain. The code is "correct" in the sense of producing ordered entries, but ...
J_H's user avatar
  • 25k
2 votes

C++ merge sort, logging, unit test, performance check

Avoid Importing Namespaces (that You Don’t Control) You have a using namespace std; declaration. It’s inside another namespace, which is a lot better than dumping ...
Davislor's user avatar
  • 7,371
1 vote

another Merge Sort

Code risks invoking undefined behavior: None of the calls to malloc() have their return values checked. ...
Harith's user avatar
  • 5,669
1 vote

another Merge Sort

Is there any other good practices that I've missed to follow in my try? Sort stability Agree with @Martin York code has stable sort concerns. A stable sort keeps the array elements in the same order ...
chux - Reinstate Monica's user avatar

Only top scored, non community-wiki answers of a minimum length are eligible