diff options
author | Jeff King <peff@peff.net> | 2023-09-19 01:33:14 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2023-09-19 01:56:15 +0300 |
commit | 1fe41944b268c5e999c5bbd9539e7dcd18c2c5de (patch) | |
tree | a9e0a32b05dd1e2a9c8492b21d49f8c7b1e2ad51 /builtin/fsmonitor--daemon.c | |
parent | 997eb910a644491b8793ae0459ee2c5d89f239f8 (diff) |
fsmonitor: mark unused hashmap callback parameters
Like many hashmap comparison functions, our cookies_cmp() does not look
at its extra void data parameter. This should have been annotated in
02c3c59e62 (hashmap: mark unused callback parameters, 2022-08-19), but
this new case was added around the same time (plus fsmonitor is not
built at all on Linux, so it is easy to miss there).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/fsmonitor--daemon.c')
-rw-r--r-- | builtin/fsmonitor--daemon.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/builtin/fsmonitor--daemon.c b/builtin/fsmonitor--daemon.c index 7c4130c981..ce511c3ed6 100644 --- a/builtin/fsmonitor--daemon.c +++ b/builtin/fsmonitor--daemon.c @@ -129,8 +129,9 @@ struct fsmonitor_cookie_item { enum fsmonitor_cookie_item_result result; }; -static int cookies_cmp(const void *data, const struct hashmap_entry *he1, - const struct hashmap_entry *he2, const void *keydata) +static int cookies_cmp(const void *data UNUSED, + const struct hashmap_entry *he1, + const struct hashmap_entry *he2, const void *keydata) { const struct fsmonitor_cookie_item *a = container_of(he1, const struct fsmonitor_cookie_item, entry); |