Currently Sorter has a number of functions which return bare pointers which are stored in a unique_ptr by the caller rather than returning a unique_ptr, and even when unique_ptr is used the objects are often allocated with new rather than make_unique.

There are also some unnecessary uses of shared_ptr that don't have any shared ownership going on.