Re: qsort() documentation

From: Warren Block <wblock_at_wonkity.com>
Date: Mon, 18 Apr 2016 12:56:24 -0600 (MDT)
On Mon, 18 Apr 2016, Hans Petter Selasky wrote:

> Hi,
>
> Are there any objections adding the following as part of documenting our 
> kernel's qsort function?
>
> Index: sys/libkern/qsort.c
> ===================================================================
> --- sys/libkern/qsort.c (revision 298202)
> +++ sys/libkern/qsort.c (working copy)
> _at__at_ -45,6 +45,10 _at__at_
>
> /*
>  * Qsort routine from Bentley & McIlroy's "Engineering a Sort Function".
> + *
> + * NOTE: This implementation of qsort() was designed to not have the

"was designed to avoid the"

> + * worst case complexity of N**2, as seen with the regular quick sort
> + * functions as described by Wikipedia.
>  */

Why Wikipedia, specifically?  There are a lot of places that describe 
quicksort.  How about just

   Note: This implementation of qsort() is designed to avoid the
   worst-case complexity of N**2 that is often seen with standard
   versions.
Received on Mon Apr 18 2016 - 16:56:31 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:04 UTC