From ac6b33a50da7d0a46e9981dcc7f6b78bd94741e8 Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Tue, 5 Mar 2019 10:06:43 -0700 Subject: [PATCH] gallium/util: add some const qualifiers in u_bitmask.c And add/update comments. Reviewed-by: Timothy Arceri Reviewed-by: Jose Fonseca --- src/gallium/auxiliary/util/u_bitmask.c | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/gallium/auxiliary/util/u_bitmask.c b/src/gallium/auxiliary/util/u_bitmask.c index 397b497d81a..433a09dd0d2 100644 --- a/src/gallium/auxiliary/util/u_bitmask.c +++ b/src/gallium/auxiliary/util/u_bitmask.c @@ -90,7 +90,7 @@ static inline boolean util_bitmask_resize(struct util_bitmask *bm, unsigned minimum_index) { - unsigned minimum_size = minimum_index + 1; + const unsigned minimum_size = minimum_index + 1; unsigned new_size; util_bitmask_word *new_words; @@ -131,7 +131,7 @@ util_bitmask_resize(struct util_bitmask *bm, /** - * Lazily update the filled. + * Check if we can increment the filled counter. */ static inline void util_bitmask_filled_set(struct util_bitmask *bm, @@ -146,6 +146,10 @@ util_bitmask_filled_set(struct util_bitmask *bm, } } + +/** + * Check if we need to decrement the filled counter. + */ static inline void util_bitmask_filled_unset(struct util_bitmask *bm, unsigned index) @@ -167,7 +171,7 @@ util_bitmask_add(struct util_bitmask *bm) assert(bm); - /* linear search for an empty index */ + /* linear search for an empty index, starting at filled position */ word = bm->filled / UTIL_BITMASK_BITS_PER_WORD; bit = bm->filled % UTIL_BITMASK_BITS_PER_WORD; mask = 1 << bit; @@ -249,9 +253,9 @@ boolean util_bitmask_get(struct util_bitmask *bm, unsigned index) { - unsigned word = index / UTIL_BITMASK_BITS_PER_WORD; - unsigned bit = index % UTIL_BITMASK_BITS_PER_WORD; - util_bitmask_word mask = 1 << bit; + const unsigned word = index / UTIL_BITMASK_BITS_PER_WORD; + const unsigned bit = index % UTIL_BITMASK_BITS_PER_WORD; + const util_bitmask_word mask = 1 << bit; assert(bm); -- 2.30.2