On Mon, Nov 25, 2013 at 5:51 PM, Rafael Zalamena <[email protected]> wrote:
> This is an implementation of the merge sort algorithm for SLIST in
> queue(3).

You gave a good description of _what_ your diff does, but not _why_ it
should be done.  The reaction of some developers is basically
"WTFityFF?!?!"  Off the top of my head, I can think of only two
reasons to include this:

1) there's more than one hand-rolled merge-sort of an SLIST in the
kernel, and this factors it out, OR
2) it's in some other project's sys/queue.h and it's turning into a
ports headache.


Philip Guenther

Reply via email to