svn commit: r247014 - head/lib/libc/stdlib

David Chisnall theraven at FreeBSD.org
Wed Feb 20 09:32:49 UTC 2013


On 20 Feb 2013, at 08:25, mdf at FreeBSD.org wrote:

> These should be declared const int *.  And the cast shouldn't be
> needed in C, since void * can be assigned to any other pointer type.

In fact, the entire function body can be replaced with:

  return (*(int*)p1 - *(int*)p2);

qsort doesn't require that you return -1, 0, or 1, it requires you return <0, 0, or >0.

David



More information about the svn-src-head mailing list