changeset 40012:975fc034aecf

array-omap, avltree-omap, rbtree-omap: Tweak style. * lib/gl_anytree_omap.h (gl_tree_nx_getput): Return 1 or 0, not true or false. * lib/gl_array_omap.c (gl_array_nx_getput): Likewise.
author Bruno Haible <bruno@clisp.org>
date Wed, 12 Dec 2018 01:03:13 +0100
parents d11da064a127
children 3f2678b62046
files ChangeLog lib/gl_anytree_omap.h lib/gl_array_omap.c
diffstat 3 files changed, 10 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Tue Dec 11 20:56:37 2018 +0100
+++ b/ChangeLog	Wed Dec 12 01:03:13 2018 +0100
@@ -1,5 +1,10 @@
 2018-12-11  Bruno Haible  <bruno@clisp.org>
 
+	array-omap, avltree-omap, rbtree-omap: Tweak style.
+	* lib/gl_anytree_omap.h (gl_tree_nx_getput): Return 1 or 0, not true or
+	false.
+	* lib/gl_array_omap.c (gl_array_nx_getput): Likewise.
+
 	rbtree-omap: Add tests.
 	* tests/test-rbtree_omap.c: New file.
 	* modules/rbtree-omap-tests: New file.
--- a/lib/gl_anytree_omap.h	Tue Dec 11 20:56:37 2018 +0100
+++ b/lib/gl_anytree_omap.h	Wed Dec 12 01:03:13 2018 +0100
@@ -129,7 +129,7 @@
     {
       if (gl_tree_nx_add_first (map, key, value) == NULL)
         return -1;
-      return true;
+      return 1;
     }
 
   compar = map->base.compar_fn;
@@ -147,7 +147,7 @@
             {
               if (gl_tree_nx_add_after (map, node, key, value) == NULL)
                 return -1;
-              return true;
+              return 1;
             }
           node = node->right;
         }
@@ -157,7 +157,7 @@
             {
               if (gl_tree_nx_add_before (map, node, key, value) == NULL)
                 return -1;
-              return true;
+              return 1;
             }
           node = node->left;
         }
@@ -165,7 +165,7 @@
         {
           *oldvaluep = node->value;
           node->value = value;
-          return false;
+          return 0;
         }
     }
 }
--- a/lib/gl_array_omap.c	Tue Dec 11 20:56:37 2018 +0100
+++ b/lib/gl_array_omap.c	Wed Dec 12 01:03:13 2018 +0100
@@ -271,7 +271,7 @@
             {
               *oldvaluep = map->pairs[mid].value;
               map->pairs[mid].value = value;
-              return false;
+              return 0;
             }
         }
       while (low < high);