diff options
author | Jeff King <peff@peff.net> | 2022-08-19 13:08:46 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-08-19 22:18:55 +0300 |
commit | 02c3c59e62a30771d209a171edfc75a4d7387ebe (patch) | |
tree | 5ba9f5cb29aa1c86d0698b7a6a51f88ccde4ab0d /hashmap.c | |
parent | 783a86c1427637d71fb2710291e677360ab5dc09 (diff) |
hashmap: mark unused callback parameters
Hashmap comparison functions must conform to a particular callback
interface, but many don't use all of their parameters. Especially the
void cmp_data pointer, but some do not use keydata either (because they
can easily form a full struct to pass when doing lookups). Let's mark
these to make -Wunused-parameter happy.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'hashmap.c')
-rw-r--r-- | hashmap.c | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -142,10 +142,10 @@ static inline struct hashmap_entry **find_entry_ptr(const struct hashmap *map, return e; } -static int always_equal(const void *unused_cmp_data, - const struct hashmap_entry *unused1, - const struct hashmap_entry *unused2, - const void *unused_keydata) +static int always_equal(const void *UNUSED(cmp_data), + const struct hashmap_entry *UNUSED(entry1), + const struct hashmap_entry *UNUSED(entry2), + const void *UNUSED(keydata)) { return 0; } @@ -313,7 +313,7 @@ struct pool_entry { unsigned char data[FLEX_ARRAY]; }; -static int pool_entry_cmp(const void *unused_cmp_data, +static int pool_entry_cmp(const void *UNUSED(cmp_data), const struct hashmap_entry *eptr, const struct hashmap_entry *entry_or_key, const void *keydata) |