diff options
| author | Jeff King <peff@peff.net> | 2022-08-19 12:08:44 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-08-19 21:18:55 +0200 |
| commit | 783a86c1427637d71fb2710291e677360ab5dc09 (patch) | |
| tree | 6301e6737288ea790d91b41036d770f7648d4d56 /t/helper | |
| parent | streaming: mark unused virtual method parameters (diff) | |
| download | git-783a86c1427637d71fb2710291e677360ab5dc09.tar.xz git-783a86c1427637d71fb2710291e677360ab5dc09.zip | |
config: mark unused callback parameters
The callback passed to git_config() must conform to a particular
interface. But most callbacks don't actually look at the extra "void
*data" parameter. Let's mark the unused parameters to make
-Wunused-parameter happy.
Note there's one unusual case here in get_remote_default() where we
actually ignore the "value" parameter. That's because it's only checking
whether the option is found at all, and not parsing its value.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/helper')
| -rw-r--r-- | t/helper/test-config.c | 2 | ||||
| -rw-r--r-- | t/helper/test-userdiff.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/t/helper/test-config.c b/t/helper/test-config.c index a6e936721f..ddd538b838 100644 --- a/t/helper/test-config.c +++ b/t/helper/test-config.c @@ -37,7 +37,7 @@ * */ -static int iterate_cb(const char *var, const char *value, void *data) +static int iterate_cb(const char *var, const char *value, void *UNUSED(data)) { static int nr; diff --git a/t/helper/test-userdiff.c b/t/helper/test-userdiff.c index f013f8a31e..64538a0c20 100644 --- a/t/helper/test-userdiff.c +++ b/t/helper/test-userdiff.c @@ -12,7 +12,7 @@ static int driver_cb(struct userdiff_driver *driver, return 0; } -static int cmd__userdiff_config(const char *var, const char *value, void *cb) +static int cmd__userdiff_config(const char *var, const char *value, void *UNUSED(cb)) { if (userdiff_config(var, value) < 0) return -1; |