diff options
| author | Jeff King <peff@peff.net> | 2023-03-28 16:57:25 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-03-28 14:11:24 -0700 |
| commit | 126e3b3d2aa06c58994ed09e12cb0a0008897039 (patch) | |
| tree | 0fd6fc68f2d3bf69aa030096db920cb0066bd548 /t/helper/test-ref-store.c | |
| parent | mark "argv" as unused when we check argc (diff) | |
| download | git-126e3b3d2aa06c58994ed09e12cb0a0008897039.tar.gz git-126e3b3d2aa06c58994ed09e12cb0a0008897039.zip | |
t/helper: mark unused argv/argc arguments
Many test helper programs do not bother to look at argc or argv, because
they don't take any options. In a user-facing program, it's a good idea
to check for unexpected arguments and complain. But for a test helper,
it's not worth the trouble to enforce this.
But we do want to tell the compiler we're OK with ignoring them, to
silence -Wunused-parameter (and obviously we can't get rid of them,
since we have to conform to the usual cmd__foo() interface).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/helper/test-ref-store.c')
| -rw-r--r-- | t/helper/test-ref-store.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/t/helper/test-ref-store.c b/t/helper/test-ref-store.c index 1745b088b7..31c79a777a 100644 --- a/t/helper/test-ref-store.c +++ b/t/helper/test-ref-store.c @@ -201,7 +201,8 @@ static int cmd_verify_ref(struct ref_store *refs, const char **argv) return ret; } -static int cmd_for_each_reflog(struct ref_store *refs, const char **argv) +static int cmd_for_each_reflog(struct ref_store *refs, + const char **argv UNUSED) { return refs_for_each_reflog(refs, each_ref, NULL); } @@ -323,7 +324,7 @@ static struct command commands[] = { { NULL, NULL } }; -int cmd__ref_store(int argc, const char **argv) +int cmd__ref_store(int argc UNUSED, const char **argv) { struct ref_store *refs; const char *func; |