]> git.ipfire.org Git - thirdparty/git.git/commitdiff
refs: mark unused parameters in do_for_each_reflog_helper()
authorJeff King <peff@peff.net>
Sat, 17 Aug 2024 08:22:22 +0000 (04:22 -0400)
committerJunio C Hamano <gitster@pobox.com>
Sat, 17 Aug 2024 16:46:11 +0000 (09:46 -0700)
This is an each_ref_fn callback, so it has to match that interface. We
marked most of these in 63e14ee2d6 (refs: mark unused each_ref_fn
parameters, 2022-08-19), but in this case:

  - this function was created in 31f898397b (refs: drop unused params
    from the reflog iterator callback, 2024-02-21), and most of the
    arguments were correctly mark as UNUSED, but "flags" was missed.

  - commit e8207717f1 (refs: add referent to each_ref_fn, 2024-08-09)
    added a new argument to the each_ref_fn callback. In most callbacks
    it added an UNUSED annotation, but it missed one case.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
refs.c

diff --git a/refs.c b/refs.c
index 74de3d30095cd5dbfabc59ae69e086365f47f3d6..584674520ee46a4d1e1914a09581691e99e45cc2 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -2390,9 +2390,10 @@ struct do_for_each_reflog_help {
        void *cb_data;
 };
 
-static int do_for_each_reflog_helper(const char *refname, const char *referent,
+static int do_for_each_reflog_helper(const char *refname,
+                                    const char *referent UNUSED,
                                     const struct object_id *oid UNUSED,
-                                    int flags,
+                                    int flags UNUSED,
                                     void *cb_data)
 {
        struct do_for_each_reflog_help *hp = cb_data;