diff options
author | Mathias Agopian <mathias@google.com> | 2009-12-03 16:14:40 -0800 |
---|---|---|
committer | Mathias Agopian <mathias@google.com> | 2009-12-03 16:14:40 -0800 |
commit | 5f53a18204ec991f5a77872806eeaa185936aa8c (patch) | |
tree | 72403af0ef4438c5d8974622bb9b98602879a7d8 | |
parent | 754c178ae551aedcbbfd3bfd1c1c3b710d9ad989 (diff) | |
download | bionic-5f53a18204ec991f5a77872806eeaa185936aa8c.zip bionic-5f53a18204ec991f5a77872806eeaa185936aa8c.tar.gz bionic-5f53a18204ec991f5a77872806eeaa185936aa8c.tar.bz2 |
Revert "Add qsort_r() implementation to the C library."android-sdk-2.1_r1android-2.1_r2.1sandroid-2.1_r2android-2.1_r1
This reverts commit 754c178ae551aedcbbfd3bfd1c1c3b710d9ad989.
Turns out we don't need it afterall (needed a stable sort anyways).
So, we'll make that change in the dev branch instead.
-rw-r--r-- | libc/Android.mk | 1 | ||||
-rw-r--r-- | libc/include/stdlib.h | 1 | ||||
-rw-r--r-- | libc/stdlib/qsort.c | 103 | ||||
-rw-r--r-- | libc/stdlib/qsort_r.c | 8 |
4 files changed, 37 insertions, 76 deletions
diff --git a/libc/Android.mk b/libc/Android.mk index c76f0c9..5718d18 100644 --- a/libc/Android.mk +++ b/libc/Android.mk @@ -154,7 +154,6 @@ libc_common_src_files := \ stdlib/nrand48.c \ stdlib/putenv.c \ stdlib/qsort.c \ - stdlib/qsort_r.c \ stdlib/seed48.c \ stdlib/setenv.c \ stdlib/setjmperr.c \ diff --git a/libc/include/stdlib.h b/libc/include/stdlib.h index 4f18833..acfe694 100644 --- a/libc/include/stdlib.h +++ b/libc/include/stdlib.h @@ -101,7 +101,6 @@ extern void * bsearch(const void *key, const void *base0, int (*compar)(const void *, const void *)); extern void qsort(void *, size_t, size_t, int (*)(const void *, const void *)); -extern void qsort_r(void *a, size_t n, size_t es, void *thunk, int (*)(void *, const void *, const void *)); extern long jrand48(unsigned short *); extern long mrand48(void); diff --git a/libc/stdlib/qsort.c b/libc/stdlib/qsort.c index 3748483..cd66961 100644 --- a/libc/stdlib/qsort.c +++ b/libc/stdlib/qsort.c @@ -1,3 +1,4 @@ +/* $OpenBSD: qsort.c,v 1.10 2005/08/08 08:05:37 espie Exp $ */ /*- * Copyright (c) 1992, 1993 * The Regents of the University of California. All rights reserved. @@ -10,7 +11,7 @@ * 2. Redistributions in binary form must reproduce the above copyright * notice, this list of conditions and the following disclaimer in the * documentation and/or other materials provided with the distribution. - * 4. Neither the name of the University nor the names of its contributors + * 3. Neither the name of the University nor the names of its contributors * may be used to endorse or promote products derived from this software * without specific prior written permission. * @@ -27,16 +28,11 @@ * SUCH DAMAGE. */ -#include <sys/cdefs.h> +#include <sys/types.h> #include <stdlib.h> -#ifdef I_AM_QSORT_R -typedef int cmp_t(void *, const void *, const void *); -#else -typedef int cmp_t(const void *, const void *); -#endif -static inline char *med3(char *, char *, char *, cmp_t *, void *); -static inline void swapfunc(char *, char *, int, int); +static __inline char *med3(char *, char *, char *, int (*)(const void *, const void *)); +static __inline void swapfunc(char *, char *, int, int); #define min(a, b) (a) < (b) ? a : b @@ -45,10 +41,10 @@ static inline void swapfunc(char *, char *, int, int); */ #define swapcode(TYPE, parmi, parmj, n) { \ long i = (n) / sizeof (TYPE); \ - TYPE *pi = (TYPE *) (parmi); \ - TYPE *pj = (TYPE *) (parmj); \ + TYPE *pi = (TYPE *) (parmi); \ + TYPE *pj = (TYPE *) (parmj); \ do { \ - TYPE t = *pi; \ + TYPE t = *pi; \ *pi++ = *pj; \ *pj++ = t; \ } while (--i > 0); \ @@ -57,12 +53,10 @@ static inline void swapfunc(char *, char *, int, int); #define SWAPINIT(a, es) swaptype = ((char *)a - (char *)0) % sizeof(long) || \ es % sizeof(long) ? 2 : es == sizeof(long)? 0 : 1; -static inline void -swapfunc(a, b, n, swaptype) - char *a, *b; - int n, swaptype; +static __inline void +swapfunc(char *a, char *b, int n, int swaptype) { - if(swaptype <= 1) + if (swaptype <= 1) swapcode(long, a, b, n) else swapcode(char, a, b, n) @@ -78,75 +72,57 @@ swapfunc(a, b, n, swaptype) #define vecswap(a, b, n) if ((n) > 0) swapfunc(a, b, n, swaptype) -#ifdef I_AM_QSORT_R -#define CMP(t, x, y) (cmp((t), (x), (y))) -#else -#define CMP(t, x, y) (cmp((x), (y))) -#endif - -static inline char * -med3(char *a, char *b, char *c, cmp_t *cmp, void *thunk -#ifndef I_AM_QSORT_R -__unused -#endif -) +static __inline char * +med3(char *a, char *b, char *c, int (*cmp)(const void *, const void *)) { - return CMP(thunk, a, b) < 0 ? - (CMP(thunk, b, c) < 0 ? b : (CMP(thunk, a, c) < 0 ? c : a )) - :(CMP(thunk, b, c) > 0 ? b : (CMP(thunk, a, c) < 0 ? a : c )); + return cmp(a, b) < 0 ? + (cmp(b, c) < 0 ? b : (cmp(a, c) < 0 ? c : a )) + :(cmp(b, c) > 0 ? b : (cmp(a, c) < 0 ? a : c )); } -#ifdef I_AM_QSORT_R -void -qsort_r(void *a, size_t n, size_t es, void *thunk, cmp_t *cmp) -#else -#define thunk NULL void -qsort(void *a, size_t n, size_t es, cmp_t *cmp) -#endif +qsort(void *aa, size_t n, size_t es, int (*cmp)(const void *, const void *)) { char *pa, *pb, *pc, *pd, *pl, *pm, *pn; - size_t d, r; - int cmp_result; - int swaptype, swap_cnt; + int d, r, swaptype, swap_cnt; + char *a = aa; loop: SWAPINIT(a, es); swap_cnt = 0; if (n < 7) { - for (pm = (char *)a + es; pm < (char *)a + n * es; pm += es) - for (pl = pm; - pl > (char *)a && CMP(thunk, pl - es, pl) > 0; + for (pm = (char *)a + es; pm < (char *) a + n * es; pm += es) + for (pl = pm; pl > (char *) a && cmp(pl - es, pl) > 0; pl -= es) swap(pl, pl - es); return; } pm = (char *)a + (n / 2) * es; if (n > 7) { - pl = a; + pl = (char *)a; pn = (char *)a + (n - 1) * es; if (n > 40) { d = (n / 8) * es; - pl = med3(pl, pl + d, pl + 2 * d, cmp, thunk); - pm = med3(pm - d, pm, pm + d, cmp, thunk); - pn = med3(pn - 2 * d, pn - d, pn, cmp, thunk); + pl = med3(pl, pl + d, pl + 2 * d, cmp); + pm = med3(pm - d, pm, pm + d, cmp); + pn = med3(pn - 2 * d, pn - d, pn, cmp); } - pm = med3(pl, pm, pn, cmp, thunk); + pm = med3(pl, pm, pn, cmp); } swap(a, pm); pa = pb = (char *)a + es; - + pc = pd = (char *)a + (n - 1) * es; for (;;) { - while (pb <= pc && (cmp_result = CMP(thunk, pb, a)) <= 0) { - if (cmp_result == 0) { + while (pb <= pc && (r = cmp(pb, a)) <= 0) { + if (r == 0) { swap_cnt = 1; swap(pa, pb); pa += es; - } + } pb += es; } - while (pb <= pc && (cmp_result = CMP(thunk, pc, a)) >= 0) { - if (cmp_result == 0) { + while (pb <= pc && (r = cmp(pc, a)) >= 0) { + if (r == 0) { swap_cnt = 1; swap(pc, pd); pd -= es; @@ -161,26 +137,21 @@ loop: SWAPINIT(a, es); pc -= es; } if (swap_cnt == 0) { /* Switch to insertion sort */ - for (pm = (char *)a + es; pm < (char *)a + n * es; pm += es) - for (pl = pm; - pl > (char *)a && CMP(thunk, pl - es, pl) > 0; + for (pm = (char *) a + es; pm < (char *) a + n * es; pm += es) + for (pl = pm; pl > (char *) a && cmp(pl - es, pl) > 0; pl -= es) swap(pl, pl - es); return; - } + } pn = (char *)a + n * es; r = min(pa - (char *)a, pb - pa); vecswap(a, pb - r, r); r = min(pd - pc, pn - pd - (int)es); vecswap(pb, pn - r, r); - if ((r = pb - pa) > es) -#ifdef I_AM_QSORT_R - qsort_r(a, r / es, es, thunk, cmp); -#else + if ((r = pb - pa) > (int)es) qsort(a, r / es, es, cmp); -#endif - if ((r = pd - pc) > es) { + if ((r = pd - pc) > (int)es) { /* Iterate rather than recurse to save stack space */ a = pn - r; n = r / es; diff --git a/libc/stdlib/qsort_r.c b/libc/stdlib/qsort_r.c deleted file mode 100644 index d57f3d3..0000000 --- a/libc/stdlib/qsort_r.c +++ /dev/null @@ -1,8 +0,0 @@ -/* - * This file is in the public domain. Originally written by Garrett - * A. Wollman. - * - * $FreeBSD: src/lib/libc/stdlib/qsort_r.c,v 1.1.36.1.2.1 2009/10/25 01:10:29 kensmith Exp $ - */ -#define I_AM_QSORT_R -#include "qsort.c" |