]> Savannah Git Hosting - gnulib.git/commitdiff
bitset: properly use false/true instead of 0/1 for Booleans
authorAkim Demaille <akim.demaille@gmail.com>
Wed, 28 Nov 2018 20:55:00 +0000 (21:55 +0100)
committerAkim Demaille <akim.demaille@gmail.com>
Thu, 29 Nov 2018 04:34:55 +0000 (05:34 +0100)
* lib/bitset/expandable.c, lib/bitset/vector.c: Use false/true, not
0/1, as Booleans.

ChangeLog
lib/bitset/expandable.c
lib/bitset/vector.c

index f3d907d99faa6852fe86a53945c875b1030269fb..bfdb8d0bc11f44dbac9961ca5eb89eaec130dfeb 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2018-11-28  Akim Demaille  <akim@lrde.epita.fr>
+
+       bitset: properly use false/true instead of 0/1 for Booleans
+       * lib/bitset/expandable.c, lib/bitset/vector.c: Use false/true, not
+       0/1, as Booleans.
+
 2018-11-28  Akim Demaille  <akim@lrde.epita.fr>
 
        bitset: rename BITSET_VARRAY as BITSET_VECTOR
index e831fdc21321568e1e04f0442b79489b44bf386a..439112aa90d7cc90df9c019b32ecdfa1dce137ef 100644 (file)
@@ -811,7 +811,7 @@ static bool
 ebitset_empty_p (bitset dst)
 {
   if (EBITSET_ZERO_P (dst))
-    return 1;
+    return true;
 
   ebitset_elts *elts = EBITSET_ELTS (dst);
   for (bitset_windex j = 0; j < EBITSET_SIZE (dst); j++)
@@ -821,7 +821,7 @@ ebitset_empty_p (bitset dst)
       if (elt)
         {
           if (!ebitset_elt_zero_p (elt))
-            return 0;
+            return false;
           /* Do some weeding as we go.  */
           ebitset_elt_remove (dst, j);
         }
@@ -830,7 +830,7 @@ ebitset_empty_p (bitset dst)
   /* All the bits are zero.  We could shrink the elts.
      For now just mark DST as known to be zero.  */
   EBITSET_ZERO_SET (dst);
-  return 1;
+  return true;
 }
 
 
index df54ee085bc72e326e48c9d6b8098a466e2733ba..3875522559446be53a3e2b83ae1354b779ba0f2d 100644 (file)
@@ -419,14 +419,12 @@ vbitset_subset_p (bitset dst, bitset src)
   unsigned i;
   for (i = 0; i < min (ssize, dsize); i++, dstp++, srcp++)
     if (*dstp != (*srcp | *dstp))
-      return 0;
+      return false;
 
   if (ssize > dsize)
-    {
-      for (; i < ssize; i++)
-        if (*srcp++)
-          return false;
-    }
+    for (; i < ssize; i++)
+      if (*srcp++)
+        return false;
 
   return true;
 }