compat: fix test for tree functions
authorRémi Denis-Courmont <remi@remlab.net>
Sun, 23 Oct 2016 17:06:30 +0000 (20:06 +0300)
committerRémi Denis-Courmont <remi@remlab.net>
Sun, 23 Oct 2016 17:06:30 +0000 (20:06 +0300)
The standard functions are tdelete, tfind, tsearch and twalk. Testing
for non-standard tdestroy does not work properly: it leads to false
negative.

compat/tdestroy.c [deleted file]
compat/tfind.c [new file with mode: 0644]
configure.ac

diff --git a/compat/tdestroy.c b/compat/tdestroy.c
deleted file mode 100644 (file)
index 63d3ddf..0000000
+++ /dev/null
@@ -1,255 +0,0 @@
-/*****************************************************************************
- * tdestroy.c : implement every t* fuctions
- *****************************************************************************/
-
-#ifdef HAVE_CONFIG_H
-# include <config.h>
-#endif
-
-/** search.h is not present so every t* functions has to be implemented */
-#ifndef HAVE_SEARCH_H
-
-#include <assert.h>
-#include <stdlib.h>
-
-typedef struct node {
-    char         *key;
-    struct node  *llink, *rlink;
-} node_t;
-
-/*     $NetBSD: tdelete.c,v 1.4 2006/03/19 01:12:08 christos Exp $     */
-
-/*
- * Tree search generalized from Knuth (6.2.2) Algorithm T just like
- * the AT&T man page says.
- *
- * The node_t structure is for internal use only, lint doesn't grok it.
- *
- * Written by reading the System V Interface Definition, not the code.
- *
- * Totally public domain.
- */
-
-/* delete node with given key */
-void *
-tdelete(vkey, vrootp, compar)
-       const void *vkey;       /* key to be deleted */
-       void      **vrootp;     /* address of the root of tree */
-       int       (*compar) (const void *, const void *);
-{
-       node_t **rootp = (node_t **)vrootp;
-       node_t *p, *q, *r;
-       int  cmp;
-
-       assert(vkey != NULL);
-       assert(compar != NULL);
-
-       if (rootp == NULL || (p = *rootp) == NULL)
-               return NULL;
-
-       while ((cmp = (*compar)(vkey, (*rootp)->key)) != 0) {
-               p = *rootp;
-               rootp = (cmp < 0) ?
-                   &(*rootp)->llink :          /* follow llink branch */
-                   &(*rootp)->rlink;           /* follow rlink branch */
-               if (*rootp == NULL)
-                       return NULL;            /* key not found */
-       }
-       r = (*rootp)->rlink;                    /* D1: */
-       if ((q = (*rootp)->llink) == NULL)      /* Left NULL? */
-               q = r;
-       else if (r != NULL) {                   /* Right link is NULL? */
-               if (r->llink == NULL) {         /* D2: Find successor */
-                       r->llink = q;
-                       q = r;
-               } else {                        /* D3: Find NULL link */
-                       for (q = r->llink; q->llink != NULL; q = r->llink)
-                               r = q;
-                       r->llink = q->rlink;
-                       q->llink = (*rootp)->llink;
-                       q->rlink = (*rootp)->rlink;
-               }
-       }
-       if (p != *rootp)
-               free(*rootp);                   /* D4: Free node */
-       *rootp = q;                             /* link parent to new node */
-       return p;
-}
-
-
-/*     $NetBSD: tdestroy.c,v 1.2 1999/09/16 11:45:37 lukem Exp $       */
-
-/*
- * Tree search generalized from Knuth (6.2.2) Algorithm T just like
- * the AT&T man page says.
- *
- * The node_t structure is for internal use only, lint doesn't grok it.
- *
- * Written by reading the System V Interface Definition, not the code.
- *
- * Totally public domain.
- */
-
-/* Walk the nodes of a tree */
-static void
-tdestroy_recurse(node_t* root, void (*free_action)(void *))
-{
-  if (root->llink != NULL)
-    tdestroy_recurse(root->llink, free_action);
-  if (root->rlink != NULL)
-    tdestroy_recurse(root->rlink, free_action);
-
-  (*free_action) ((void *) root->key);
-  free(root);
-}
-
-void
-tdestroy(vrootp, freefct)
-       void *vrootp;
-       void (*freefct)(void *);
-{
-  node_t *root = (node_t *) vrootp;
-
-  if (root != NULL)
-    tdestroy_recurse(root, freefct);
-}
-
-
-/*     $NetBSD: tfind.c,v 1.5 2005/03/23 08:16:53 kleink Exp $ */
-
-/*
- * Tree search generalized from Knuth (6.2.2) Algorithm T just like
- * the AT&T man page says.
- *
- * The node_t structure is for internal use only, lint doesn't grok it.
- *
- * Written by reading the System V Interface Definition, not the code.
- *
- * Totally public domain.
- */
-
-/* find a node, or return 0 */
-void *
-tfind(vkey, vrootp, compar)
-       const void *vkey;               /* key to be found */
-       const void **vrootp;            /* address of the tree root */
-       int (*compar) (const void *, const void *);
-{
-       node_t * const *rootp = (node_t * const*)vrootp;
-
-       assert(vkey != NULL);
-       assert(compar != NULL);
-
-       if (rootp == NULL)
-               return NULL;
-
-       while (*rootp != NULL) {                /* T1: */
-               int r;
-
-               if ((r = (*compar)(vkey, (*rootp)->key)) == 0)  /* T2: */
-                       return *rootp;          /* key found */
-               rootp = (r < 0) ?
-                   &(*rootp)->llink :          /* T3: follow left branch */
-                   &(*rootp)->rlink;           /* T4: follow right branch */
-       }
-       return NULL;
-}
-
-
-/*     $NetBSD: tsearch.c,v 1.5 2005/11/29 03:12:00 christos Exp $     */
-
-/*
- * Tree search generalized from Knuth (6.2.2) Algorithm T just like
- * the AT&T man page says.
- *
- * The node_t structure is for internal use only, lint doesn't grok it.
- *
- * Written by reading the System V Interface Definition, not the code.
- *
- * Totally public domain.
- */
-
-/* find or insert datum into search tree */
-void *
-tsearch(vkey, vrootp, compar)
-       const void *vkey;               /* key to be located */
-       void **vrootp;                  /* address of tree root */
-       int (*compar) (const void *, const void *);
-{
-       node_t *q;
-       node_t **rootp = (node_t **)vrootp;
-
-       assert(vkey != NULL);
-       assert(compar != NULL);
-
-       if (rootp == NULL)
-               return NULL;
-
-       while (*rootp != NULL) {        /* Knuth's T1: */
-               int r;
-
-               if ((r = (*compar)(vkey, (*rootp)->key)) == 0)  /* T2: */
-                       return *rootp;          /* we found it! */
-
-               rootp = (r < 0) ?
-                   &(*rootp)->llink :          /* T3: follow left branch */
-                   &(*rootp)->rlink;           /* T4: follow right branch */
-       }
-
-       q = malloc(sizeof(node_t));             /* T5: key not found */
-       if (q != 0) {                           /* make new node */
-               *rootp = q;                     /* link new node to old */
-               q->key = (void*)vkey;   /* initialize new node */
-               q->llink = q->rlink = NULL;
-       }
-       return q;
-}
-
-
-/*     $NetBSD: twalk.c,v 1.2 1999/09/16 11:45:37 lukem Exp $  */
-
-/*
- * Tree search generalized from Knuth (6.2.2) Algorithm T just like
- * the AT&T man page says.
- *
- * The node_t structure is for internal use only, lint doesn't grok it.
- *
- * Written by reading the System V Interface Definition, not the code.
- *
- * Totally public domain.
- */
-
-/* Walk the nodes of a tree */
-static void
-twalk_recurse(root, action, level)
-       const node_t *root;     /* Root of the tree to be walked */
-       void (*action) (const void *, VISIT, int);
-       int level;
-{
-       assert(root != NULL);
-       assert(action != NULL);
-
-       if (root->llink == NULL && root->rlink == NULL)
-               (*action)(root, leaf, level);
-       else {
-               (*action)(root, preorder, level);
-               if (root->llink != NULL)
-                       twalk_recurse(root->llink, action, level + 1);
-               (*action)(root, postorder, level);
-               if (root->rlink != NULL)
-                       twalk_recurse(root->rlink, action, level + 1);
-               (*action)(root, endorder, level);
-       }
-}
-
-/* Walk the nodes of a tree */
-void
-twalk(vroot, action)
-       const void *vroot;      /* Root of the tree to be walked */
-       void (*action) (const void *, VISIT, int);
-{
-       if (vroot != NULL && action != NULL)
-               twalk_recurse(vroot, action, 0);
-}
-
-#endif // HAVE_SEARCH_H
diff --git a/compat/tfind.c b/compat/tfind.c
new file mode 100644 (file)
index 0000000..804ecb8
--- /dev/null
@@ -0,0 +1,255 @@
+/*****************************************************************************
+ * tfind.c : implement every t* fuctions
+ *****************************************************************************/
+
+#ifdef HAVE_CONFIG_H
+# include <config.h>
+#endif
+
+/** search.h is not present so every t* functions has to be implemented */
+#ifndef HAVE_SEARCH_H
+
+#include <assert.h>
+#include <stdlib.h>
+
+typedef struct node {
+    char         *key;
+    struct node  *llink, *rlink;
+} node_t;
+
+/*     $NetBSD: tdelete.c,v 1.4 2006/03/19 01:12:08 christos Exp $     */
+
+/*
+ * Tree search generalized from Knuth (6.2.2) Algorithm T just like
+ * the AT&T man page says.
+ *
+ * The node_t structure is for internal use only, lint doesn't grok it.
+ *
+ * Written by reading the System V Interface Definition, not the code.
+ *
+ * Totally public domain.
+ */
+
+/* delete node with given key */
+void *
+tdelete(vkey, vrootp, compar)
+       const void *vkey;       /* key to be deleted */
+       void      **vrootp;     /* address of the root of tree */
+       int       (*compar) (const void *, const void *);
+{
+       node_t **rootp = (node_t **)vrootp;
+       node_t *p, *q, *r;
+       int  cmp;
+
+       assert(vkey != NULL);
+       assert(compar != NULL);
+
+       if (rootp == NULL || (p = *rootp) == NULL)
+               return NULL;
+
+       while ((cmp = (*compar)(vkey, (*rootp)->key)) != 0) {
+               p = *rootp;
+               rootp = (cmp < 0) ?
+                   &(*rootp)->llink :          /* follow llink branch */
+                   &(*rootp)->rlink;           /* follow rlink branch */
+               if (*rootp == NULL)
+                       return NULL;            /* key not found */
+       }
+       r = (*rootp)->rlink;                    /* D1: */
+       if ((q = (*rootp)->llink) == NULL)      /* Left NULL? */
+               q = r;
+       else if (r != NULL) {                   /* Right link is NULL? */
+               if (r->llink == NULL) {         /* D2: Find successor */
+                       r->llink = q;
+                       q = r;
+               } else {                        /* D3: Find NULL link */
+                       for (q = r->llink; q->llink != NULL; q = r->llink)
+                               r = q;
+                       r->llink = q->rlink;
+                       q->llink = (*rootp)->llink;
+                       q->rlink = (*rootp)->rlink;
+               }
+       }
+       if (p != *rootp)
+               free(*rootp);                   /* D4: Free node */
+       *rootp = q;                             /* link parent to new node */
+       return p;
+}
+
+
+/*     $NetBSD: tdestroy.c,v 1.2 1999/09/16 11:45:37 lukem Exp $       */
+
+/*
+ * Tree search generalized from Knuth (6.2.2) Algorithm T just like
+ * the AT&T man page says.
+ *
+ * The node_t structure is for internal use only, lint doesn't grok it.
+ *
+ * Written by reading the System V Interface Definition, not the code.
+ *
+ * Totally public domain.
+ */
+
+/* Walk the nodes of a tree */
+static void
+tdestroy_recurse(node_t* root, void (*free_action)(void *))
+{
+  if (root->llink != NULL)
+    tdestroy_recurse(root->llink, free_action);
+  if (root->rlink != NULL)
+    tdestroy_recurse(root->rlink, free_action);
+
+  (*free_action) ((void *) root->key);
+  free(root);
+}
+
+void
+tdestroy(vrootp, freefct)
+       void *vrootp;
+       void (*freefct)(void *);
+{
+  node_t *root = (node_t *) vrootp;
+
+  if (root != NULL)
+    tdestroy_recurse(root, freefct);
+}
+
+
+/*     $NetBSD: tfind.c,v 1.5 2005/03/23 08:16:53 kleink Exp $ */
+
+/*
+ * Tree search generalized from Knuth (6.2.2) Algorithm T just like
+ * the AT&T man page says.
+ *
+ * The node_t structure is for internal use only, lint doesn't grok it.
+ *
+ * Written by reading the System V Interface Definition, not the code.
+ *
+ * Totally public domain.
+ */
+
+/* find a node, or return 0 */
+void *
+tfind(vkey, vrootp, compar)
+       const void *vkey;               /* key to be found */
+       const void **vrootp;            /* address of the tree root */
+       int (*compar) (const void *, const void *);
+{
+       node_t * const *rootp = (node_t * const*)vrootp;
+
+       assert(vkey != NULL);
+       assert(compar != NULL);
+
+       if (rootp == NULL)
+               return NULL;
+
+       while (*rootp != NULL) {                /* T1: */
+               int r;
+
+               if ((r = (*compar)(vkey, (*rootp)->key)) == 0)  /* T2: */
+                       return *rootp;          /* key found */
+               rootp = (r < 0) ?
+                   &(*rootp)->llink :          /* T3: follow left branch */
+                   &(*rootp)->rlink;           /* T4: follow right branch */
+       }
+       return NULL;
+}
+
+
+/*     $NetBSD: tsearch.c,v 1.5 2005/11/29 03:12:00 christos Exp $     */
+
+/*
+ * Tree search generalized from Knuth (6.2.2) Algorithm T just like
+ * the AT&T man page says.
+ *
+ * The node_t structure is for internal use only, lint doesn't grok it.
+ *
+ * Written by reading the System V Interface Definition, not the code.
+ *
+ * Totally public domain.
+ */
+
+/* find or insert datum into search tree */
+void *
+tsearch(vkey, vrootp, compar)
+       const void *vkey;               /* key to be located */
+       void **vrootp;                  /* address of tree root */
+       int (*compar) (const void *, const void *);
+{
+       node_t *q;
+       node_t **rootp = (node_t **)vrootp;
+
+       assert(vkey != NULL);
+       assert(compar != NULL);
+
+       if (rootp == NULL)
+               return NULL;
+
+       while (*rootp != NULL) {        /* Knuth's T1: */
+               int r;
+
+               if ((r = (*compar)(vkey, (*rootp)->key)) == 0)  /* T2: */
+                       return *rootp;          /* we found it! */
+
+               rootp = (r < 0) ?
+                   &(*rootp)->llink :          /* T3: follow left branch */
+                   &(*rootp)->rlink;           /* T4: follow right branch */
+       }
+
+       q = malloc(sizeof(node_t));             /* T5: key not found */
+       if (q != 0) {                           /* make new node */
+               *rootp = q;                     /* link new node to old */
+               q->key = (void*)vkey;   /* initialize new node */
+               q->llink = q->rlink = NULL;
+       }
+       return q;
+}
+
+
+/*     $NetBSD: twalk.c,v 1.2 1999/09/16 11:45:37 lukem Exp $  */
+
+/*
+ * Tree search generalized from Knuth (6.2.2) Algorithm T just like
+ * the AT&T man page says.
+ *
+ * The node_t structure is for internal use only, lint doesn't grok it.
+ *
+ * Written by reading the System V Interface Definition, not the code.
+ *
+ * Totally public domain.
+ */
+
+/* Walk the nodes of a tree */
+static void
+twalk_recurse(root, action, level)
+       const node_t *root;     /* Root of the tree to be walked */
+       void (*action) (const void *, VISIT, int);
+       int level;
+{
+       assert(root != NULL);
+       assert(action != NULL);
+
+       if (root->llink == NULL && root->rlink == NULL)
+               (*action)(root, leaf, level);
+       else {
+               (*action)(root, preorder, level);
+               if (root->llink != NULL)
+                       twalk_recurse(root->llink, action, level + 1);
+               (*action)(root, postorder, level);
+               if (root->rlink != NULL)
+                       twalk_recurse(root->rlink, action, level + 1);
+               (*action)(root, endorder, level);
+       }
+}
+
+/* Walk the nodes of a tree */
+void
+twalk(vroot, action)
+       const void *vroot;      /* Root of the tree to be walked */
+       void (*action) (const void *, VISIT, int);
+{
+       if (vroot != NULL && action != NULL)
+               twalk_recurse(vroot, action, 0);
+}
+
+#endif // HAVE_SEARCH_H
index 900286c..98c209d 100644 (file)
@@ -599,7 +599,7 @@ need_libc=false
 dnl Check for usual libc functions
 AC_CHECK_DECLS([nanosleep],,,[#include <time.h>])
 AC_CHECK_FUNCS([daemon fcntl flock fstatvfs fork getenv getpwuid_r isatty lstat memalign mkostemp mmap open_memstream openat pread posix_fadvise posix_madvise setlocale stricmp strnicmp strptime uselocale pthread_cond_timedwait_monotonic_np pthread_condattr_setclock])
-AC_REPLACE_FUNCS([atof atoll dirfd fdopendir ffsll flockfile fsync getdelim getpid lldiv memrchr nrand48 poll posix_memalign recvmsg rewind sendmsg setenv strcasecmp strcasestr strdup strlcpy strndup strnlen strnstr strsep strtof strtok_r strtoll swab tdestroy timegm timespec_get strverscmp])
+AC_REPLACE_FUNCS([atof atoll dirfd fdopendir ffsll flockfile fsync getdelim getpid lldiv memrchr nrand48 poll posix_memalign recvmsg rewind sendmsg setenv strcasecmp strcasestr strdup strlcpy strndup strnlen strnstr strsep strtof strtok_r strtoll swab tfind timegm timespec_get strverscmp])
 AC_REPLACE_FUNCS([gettimeofday])
 AC_CHECK_FUNC(fdatasync,,
   [AC_DEFINE(fdatasync, fsync, [Alias fdatasync() to fsync() if missing.])