Mesa (master): gallium/hash_table: use the same callback signatures as util/hash_table
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Feb 26 21:14:03 UTC 2020
Module: Mesa
Branch: master
Commit: 10d235a84319ed4137fe0c6d22d8bb9a52d3174a
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=10d235a84319ed4137fe0c6d22d8bb9a52d3174a
Author: Marek Olšák <marek.olsak at amd.com>
Date: Wed Feb 5 14:19:40 2020 -0500
gallium/hash_table: use the same callback signatures as util/hash_table
Reviewed-by: Kristian H. Kristensen <hoegsberg at google.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/3722>
---
src/gallium/auxiliary/util/u_hash_table.c | 38 ++++++++++++++--------------
src/gallium/auxiliary/util/u_hash_table.h | 6 ++---
src/gallium/state_trackers/nine/nine_ff.c | 18 ++++++-------
src/gallium/state_trackers/nine/nine_pdata.h | 12 ++++-----
src/gallium/winsys/svga/drm/vmw_screen.c | 6 ++---
5 files changed, 40 insertions(+), 40 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_hash_table.c b/src/gallium/auxiliary/util/u_hash_table.c
index e58c7823be2..0db5ddbee5e 100644
--- a/src/gallium/auxiliary/util/u_hash_table.c
+++ b/src/gallium/auxiliary/util/u_hash_table.c
@@ -57,10 +57,10 @@ struct util_hash_table
struct cso_hash cso;
/** Hash function */
- unsigned (*hash)(void *key);
+ uint32_t (*hash)(const void *key);
/** Compare two keys */
- int (*compare)(void *key1, void *key2);
+ bool (*equal)(const void *key1, const void *key2);
/* TODO: key, value destructors? */
};
@@ -81,8 +81,8 @@ util_hash_table_item(struct cso_hash_iter iter)
struct util_hash_table *
-util_hash_table_create(unsigned (*hash)(void *key),
- int (*compare)(void *key1, void *key2))
+util_hash_table_create(uint32_t (*hash)(const void *key),
+ bool (*equal)(const void *key1, const void *key2))
{
struct util_hash_table *ht;
@@ -93,34 +93,34 @@ util_hash_table_create(unsigned (*hash)(void *key),
cso_hash_init(&ht->cso);
ht->hash = hash;
- ht->compare = compare;
+ ht->equal = equal;
return ht;
}
-static unsigned
-pointer_hash(void *key)
+static uint32_t
+pointer_hash(const void *key)
{
return _mesa_hash_pointer(key);
}
-static int
-pointer_compare(void *a, void *b)
+static bool
+pointer_equal(const void *a, const void *b)
{
- return a != b;
+ return a == b;
}
struct util_hash_table *
util_hash_table_create_ptr_keys(void)
{
- return util_hash_table_create(pointer_hash, pointer_compare);
+ return util_hash_table_create(pointer_hash, pointer_equal);
}
-static unsigned hash_fd(void *key)
+static uint32_t hash_fd(const void *key)
{
#if DETECT_OS_UNIX
int fd = pointer_to_intptr(key);
@@ -135,7 +135,7 @@ static unsigned hash_fd(void *key)
}
-static int compare_fd(void *key1, void *key2)
+static bool equal_fd(const void *key1, const void *key2)
{
#if DETECT_OS_UNIX
int fd1 = pointer_to_intptr(key1);
@@ -145,9 +145,9 @@ static int compare_fd(void *key1, void *key2)
fstat(fd1, &stat1);
fstat(fd2, &stat2);
- return stat1.st_dev != stat2.st_dev ||
- stat1.st_ino != stat2.st_ino ||
- stat1.st_rdev != stat2.st_rdev;
+ return stat1.st_dev == stat2.st_dev &&
+ stat1.st_ino == stat2.st_ino &&
+ stat1.st_rdev == stat2.st_rdev;
#else
return 0;
#endif
@@ -157,7 +157,7 @@ static int compare_fd(void *key1, void *key2)
struct util_hash_table *
util_hash_table_create_fd_keys(void)
{
- return util_hash_table_create(hash_fd, compare_fd);
+ return util_hash_table_create(hash_fd, equal_fd);
}
@@ -172,7 +172,7 @@ util_hash_table_find_iter(struct util_hash_table *ht,
iter = cso_hash_find(&ht->cso, key_hash);
while (!cso_hash_iter_is_null(iter)) {
item = (struct util_hash_table_item *)cso_hash_iter_data(iter);
- if (!ht->compare(item->key, key))
+ if (ht->equal(item->key, key))
break;
iter = cso_hash_iter_next(iter);
}
@@ -192,7 +192,7 @@ util_hash_table_find_item(struct util_hash_table *ht,
iter = cso_hash_find(&ht->cso, key_hash);
while (!cso_hash_iter_is_null(iter)) {
item = (struct util_hash_table_item *)cso_hash_iter_data(iter);
- if (!ht->compare(item->key, key))
+ if (ht->equal(item->key, key))
return item;
iter = cso_hash_iter_next(iter);
}
diff --git a/src/gallium/auxiliary/util/u_hash_table.h b/src/gallium/auxiliary/util/u_hash_table.h
index 7db87c4367b..3d751f423aa 100644
--- a/src/gallium/auxiliary/util/u_hash_table.h
+++ b/src/gallium/auxiliary/util/u_hash_table.h
@@ -53,11 +53,11 @@ struct util_hash_table;
* Create an hash table.
*
* @param hash hash function
- * @param compare should return 0 for two equal keys.
+ * @param equal should return true for two equal keys.
*/
struct util_hash_table *
-util_hash_table_create(unsigned (*hash)(void *key),
- int (*compare)(void *key1, void *key2));
+util_hash_table_create(uint32_t (*hash)(const void *key),
+ bool (*equal)(const void *key1, const void *key2));
/**
* Create a hash table where the keys are generic pointers.
diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c
index 5e756b36707..8973dcbbc28 100644
--- a/src/gallium/state_trackers/nine/nine_ff.c
+++ b/src/gallium/state_trackers/nine/nine_ff.c
@@ -124,7 +124,7 @@ struct nine_ff_ps_key
};
};
-static unsigned nine_ff_vs_key_hash(void *key)
+static uint32_t nine_ff_vs_key_hash(const void *key)
{
struct nine_ff_vs_key *vs = key;
unsigned i;
@@ -133,14 +133,14 @@ static unsigned nine_ff_vs_key_hash(void *key)
hash ^= vs->value32[i];
return hash;
}
-static int nine_ff_vs_key_comp(void *key1, void *key2)
+static bool nine_ff_vs_key_comp(const void *key1, const void *key2)
{
struct nine_ff_vs_key *a = (struct nine_ff_vs_key *)key1;
struct nine_ff_vs_key *b = (struct nine_ff_vs_key *)key2;
- return memcmp(a->value64, b->value64, sizeof(a->value64));
+ return memcmp(a->value64, b->value64, sizeof(a->value64)) == 0;
}
-static unsigned nine_ff_ps_key_hash(void *key)
+static uint32_t nine_ff_ps_key_hash(const void *key)
{
struct nine_ff_ps_key *ps = key;
unsigned i;
@@ -149,20 +149,20 @@ static unsigned nine_ff_ps_key_hash(void *key)
hash ^= ps->value32[i];
return hash;
}
-static int nine_ff_ps_key_comp(void *key1, void *key2)
+static bool nine_ff_ps_key_comp(const void *key1, const void *key2)
{
struct nine_ff_ps_key *a = (struct nine_ff_ps_key *)key1;
struct nine_ff_ps_key *b = (struct nine_ff_ps_key *)key2;
- return memcmp(a->value64, b->value64, sizeof(a->value64));
+ return memcmp(a->value64, b->value64, sizeof(a->value64)) == 0;
}
-static unsigned nine_ff_fvf_key_hash(void *key)
+static uint32_t nine_ff_fvf_key_hash(const void *key)
{
return *(DWORD *)key;
}
-static int nine_ff_fvf_key_comp(void *key1, void *key2)
+static bool nine_ff_fvf_key_comp(const void *key1, const void *key2)
{
- return *(DWORD *)key1 != *(DWORD *)key2;
+ return *(DWORD *)key1 == *(DWORD *)key2;
}
static void nine_ff_prune_vs(struct NineDevice9 *);
diff --git a/src/gallium/state_trackers/nine/nine_pdata.h b/src/gallium/state_trackers/nine/nine_pdata.h
index 92e50c8a72c..8c73cd619ba 100644
--- a/src/gallium/state_trackers/nine/nine_pdata.h
+++ b/src/gallium/state_trackers/nine/nine_pdata.h
@@ -9,15 +9,15 @@ struct pheader
DWORD size;
};
-static int
-ht_guid_compare( void *a,
- void *b )
+static bool
+ht_guid_compare( const void *a,
+ const void *b )
{
- return GUID_equal(a, b) ? 0 : 1;
+ return GUID_equal(a, b);
}
-static unsigned
-ht_guid_hash( void *key )
+static uint32_t
+ht_guid_hash( const void *key )
{
unsigned i, hash = 0;
const unsigned char *str = key;
diff --git a/src/gallium/winsys/svga/drm/vmw_screen.c b/src/gallium/winsys/svga/drm/vmw_screen.c
index e3440bfd957..5f68a894673 100644
--- a/src/gallium/winsys/svga/drm/vmw_screen.c
+++ b/src/gallium/winsys/svga/drm/vmw_screen.c
@@ -43,13 +43,13 @@
static struct util_hash_table *dev_hash = NULL;
-static int vmw_dev_compare(void *key1, void *key2)
+static bool vmw_dev_compare(const void *key1, const void *key2)
{
return (major(*(dev_t *)key1) == major(*(dev_t *)key2) &&
- minor(*(dev_t *)key1) == minor(*(dev_t *)key2)) ? 0 : 1;
+ minor(*(dev_t *)key1) == minor(*(dev_t *)key2));
}
-static unsigned vmw_dev_hash(void *key)
+static uint32_t vmw_dev_hash(const void *key)
{
return (major(*(dev_t *) key) << 16) | minor(*(dev_t *) key);
}
More information about the mesa-commit
mailing list