X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=bertos%2Fstruct%2Fbitarray.h;h=3e96b7c5f92e2c9d86a8cc588863b7e4247e9f29;hb=8ad4f1f6d610ce74466f049b25950cf0fd981856;hp=3bbaa8d19cf277b96037f0da96ee4eb4fb76c33a;hpb=f56cf80247f6fbe18ddd34357ee306107fffeca0;p=bertos.git

diff --git a/bertos/struct/bitarray.h b/bertos/struct/bitarray.h
index 3bbaa8d1..3e96b7c5 100644
--- a/bertos/struct/bitarray.h
+++ b/bertos/struct/bitarray.h
@@ -47,13 +47,13 @@
 
 typedef struct BitArray
 {
-	size_t size;
-	size_t bitarray_len;
-	uint8_t *array;
+	size_t size;          /// Size in bytes of the bitarray
+	size_t bitarray_len;  /// Number of bits used
+	uint8_t *array;       /// Pointer to memory occupied by the bitarray
 } BitArray;
 
-#define ALLOC_BITARRAY(name, size)   uint8_t name[DIV_ROUNDUP((size), 8)]
-#define BIT_ARRAY_SIZE(name)         (sizeof((name)) * 8)
+#define BITARRAY_ALLOC(name, size)   uint8_t name[DIV_ROUNDUP((size), 8)]
+#define BITARRAY_SIZE(name)         (sizeof((name)) * 8)
 
 INLINE void bitarray_set(BitArray *bitx, int idx)
 {
@@ -75,7 +75,7 @@ INLINE void bitarray_clear(BitArray *bitx, int idx)
 	bitx->array[page] &= ~BV(bit);
 }
 
-INLINE void bitarray_setOffset(BitArray *bitx, int idx, int offset)
+INLINE void bitarray_setRange(BitArray *bitx, int idx, int offset)
 {
 	ASSERT((size_t)idx <= bitx->bitarray_len);
 
@@ -84,7 +84,7 @@ INLINE void bitarray_setOffset(BitArray *bitx, int idx, int offset)
 }
 
 
-INLINE void bitarray_clearOffset(BitArray *bitx, int idx, int offset)
+INLINE void bitarray_clearRange(BitArray *bitx, int idx, int offset)
 {
 	ASSERT((size_t)idx <= bitx->bitarray_len);
 
@@ -92,7 +92,7 @@ INLINE void bitarray_clearOffset(BitArray *bitx, int idx, int offset)
 		bitarray_clear(bitx, i);
 }
 
-INLINE bool bitarray_check(BitArray *bitx, int idx)
+INLINE bool bitarray_test(BitArray *bitx, int idx)
 {
 	ASSERT((size_t)idx <= bitx->bitarray_len);
 	int page = idx / 8;
@@ -101,28 +101,23 @@ INLINE bool bitarray_check(BitArray *bitx, int idx)
 	return (bitx->array[page] & BV(bit));
 }
 
-/*
- * Ugly!.. reformat it.
+/**
+ * Check if the bitarray is full
+ *
+ * Only \a bitarray_len bits are tested.
  */
-INLINE bool bitarray_full(BitArray *bitx)
+INLINE bool bitarray_isFull(BitArray *bitx)
 {
-	int count = bitx->size;
-	for (size_t page = 0; page <= bitx->size / 8; page++)
+	// test full bytes except the last one
+	for (size_t page = 0; page <= bitx->size - 2; page++)
 	{
-		if (count < 8)
-		{
-			for (size_t i = page * 8; i <= bitx->bitarray_len; i++)
-				if (!bitarray_check(bitx, i))
-					return 0;
-				count--;
-		}
-		else
-		{
-			if (!(bitx->array[page] == 0xff))
-				return 0;
-		}
-		count -= 8;
+		if (!(bitx->array[page] == 0xff))
+			return 0;
 	}
+	// test the last byte using the correct bitmask
+	uint8_t mask = BV(bitx->bitarray_len >> 3) - 1;
+	if (!(bitx->array[bitx->size - 1] & mask))
+		return 0;
 
 	return 1;
 }
@@ -130,12 +125,12 @@ INLINE bool bitarray_full(BitArray *bitx)
 /*
  * Ugly!.. reformat it.
  */
-INLINE bool bitarray_blockFull(BitArray *bitx, int idx, int offset)
+INLINE bool bitarray_isRangeFull(BitArray *bitx, int idx, int offset)
 {
 	ASSERT((size_t)(idx + offset) <= bitx->bitarray_len);
 
 	for (int i = idx; i <= idx + offset; i++)
-		if (!bitarray_check(bitx, i))
+		if (!bitarray_test(bitx, i))
 			return 0;
 
 	return 1;
@@ -144,12 +139,12 @@ INLINE bool bitarray_blockFull(BitArray *bitx, int idx, int offset)
 /*
  * Ugly!.. reformat it.
  */
-INLINE bool bitarray_blockEmpty(BitArray *bitx, int idx, int offset)
+INLINE bool bitarray_isRangeEmpty(BitArray *bitx, int idx, int offset)
 {
 	ASSERT((size_t)(idx + offset) <= bitx->bitarray_len);
 
 	for (int i = idx; i <= idx + offset; i++)
-		if (bitarray_check(bitx, i))
+		if (bitarray_test(bitx, i))
 			return 0;
 
 	return 1;
@@ -166,7 +161,7 @@ INLINE void bitarray_dump(BitArray *bitx)
 
 	while (count--)
 	{
-		kprintf("%d", bitarray_check(bitx, i++));
+		kprintf("%d", bitarray_test(bitx, i++));
 		if (j == 7)
 		{
 			kprintf("..%02x [%d]\n", bitx->array[(i / 8) - 1], i);