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>

diff --git a/archive-tar.c b/archive-tar.c
index 3d77e0f75095743d3da9874832e2f8bdccf5e896..45e5d9140765e27c5cb6ceec09914fd4967e9b0c 100644 (file)
--- a/archive-tar.c
+++ b/archive-tar.c
@@ -366,7 +366,8 @@ static struct archiver *find_tar_filter(const char *name, size_t len)
        return NULL;
 }
 
-static int tar_filter_config(const char *var, const char *value, void *data)
+static int tar_filter_config(const char *var, const char *value,
+                            void *UNUSED(data))
 {
        struct archiver *ar;
        const char *name;

diff --git a/archive-zip.c b/archive-zip.c
index 9fe43d740d83f99111c31ef64207cf908765c17e..854bceb01872adcdfdf940db82001d52e1fb0498 100644 (file)
--- a/archive-zip.c
+++ b/archive-zip.c
@@ -612,7 +612,8 @@ static void dos_time(timestamp_t *timestamp, int *dos_date, int *dos_time)
        *dos_time = tm.tm_sec / 2 + tm.tm_min * 32 + tm.tm_hour * 2048;
 }
 
-static int archive_zip_config(const char *var, const char *value, void *data)
+static int archive_zip_config(const char *var, const char *value,
+                             void *UNUSED(data))
 {
        return userdiff_config(var, value);
 }

diff --git a/builtin/am.c b/builtin/am.c
index 93bec62afa993cb236ce09c13b38932ee396f250..0811b9ff676f531c48e5d8b6875f1f77f83bc439 100644 (file)
--- a/builtin/am.c
+++ b/builtin/am.c
@@ -2301,7 +2301,7 @@ static int parse_opt_show_current_patch(const struct option *opt, const char *ar
        return 0;
 }
 
-static int git_am_config(const char *k, const char *v, void *cb)
+static int git_am_config(const char *k, const char *v, void *UNUSED(cb))
 {
        int status;
 

diff --git a/builtin/commit-graph.c b/builtin/commit-graph.c
index 51c4040ea6c879bda3631343643f3a52063a1bdc..ea923ea33ad73a24a812e17410f563d657814295 100644 (file)
--- a/builtin/commit-graph.c
+++ b/builtin/commit-graph.c
@@ -179,7 +179,7 @@ static int write_option_max_new_filters(const struct option *opt,
 }
 
 static int git_commit_graph_write_config(const char *var, const char *value,
-                                        void *cb)
+                                        void *UNUSED(cb))
 {
        if (!strcmp(var, "commitgraph.maxnewfilters"))
                write_opts.max_new_filters = git_config_int(var, value);

diff --git a/builtin/config.c b/builtin/config.c
index e7b88a9c08dc17055dd201eb5f985a77b6150604..bdc8b1d1a89771a9eec76ec5a2ef68f87dc3bde0 100644 (file)
--- a/builtin/config.c
+++ b/builtin/config.c
@@ -207,7 +207,8 @@ static void show_config_scope(struct strbuf *buf)
        strbuf_addch(buf, term);
 }
 
-static int show_all_config(const char *key_, const char *value_, void *cb)
+static int show_all_config(const char *key_, const char *value_,
+                          void *UNUSED(cb))
 {
        if (show_origin || show_scope) {
                struct strbuf buf = STRBUF_INIT;
@@ -458,7 +459,8 @@ static const char *get_color_slot;
 static const char *get_colorbool_slot;
 static char parsed_color[COLOR_MAXLEN];
 
-static int git_get_color_config(const char *var, const char *value, void *cb)
+static int git_get_color_config(const char *var, const char *value,
+                               void *UNUSED(cb))
 {
        if (!strcmp(var, get_color_slot)) {
                if (!value)
@@ -490,7 +492,7 @@ static int get_colorbool_found;
 static int get_diff_color_found;
 static int get_color_ui_found;
 static int git_get_colorbool_config(const char *var, const char *value,
-               void *cb)
+                                   void *UNUSED(data))
 {
        if (!strcmp(var, get_colorbool_slot))
                get_colorbool_found = git_config_colorbool(var, value);

diff --git a/builtin/multi-pack-index.c b/builtin/multi-pack-index.c
index 8f24d59a7537828c2bcf52c541d2a75e0d521d7c..8d156766afe86cefa54de7013c1ab353dce800f3 100644 (file)
--- a/builtin/multi-pack-index.c
+++ b/builtin/multi-pack-index.c
@@ -78,7 +78,7 @@ static struct option *add_common_options(struct option *prev)
 }
 
 static int git_multi_pack_index_write_config(const char *var, const char *value,
-                                            void *cb)
+                                            void *UNUSED(cb))
 {
        if (!strcmp(var, "pack.writebitmaphashcache")) {
                if (git_config_bool(var, value))

diff --git a/builtin/remote.c b/builtin/remote.c
index b390360f079ec214126b8544acf5a2c62d42be2a..87dda7c37bdb984a090c237e258c78f85c762f3f 100644 (file)
--- a/builtin/remote.c
+++ b/builtin/remote.c
@@ -1486,7 +1486,7 @@ static int prune(int argc, const char **argv)
        return result;
 }
 
-static int get_remote_default(const char *key, const char *value, void *priv)
+static int get_remote_default(const char *key, const char *UNUSED(value), void *priv)
 {
        if (strcmp(key, "remotes.default") == 0) {
                int *found = priv;

diff --git a/color.c b/color.c
index 4f884c6b3dc1d98c99d3b5e5be0e990f58434d8e..04ad0a8bf75375a8f3556c925a8fd4a57e61c29b 100644 (file)
--- a/color.c
+++ b/color.c
@@ -415,7 +415,7 @@ int want_color_fd(int fd, int var)
        return var;
 }
 
-int git_color_config(const char *var, const char *value, void *cb)
+int git_color_config(const char *var, const char *value, void *UNUSED(cb))
 {
        if (!strcmp(var, "color.ui")) {
                git_use_color_default = git_config_colorbool(var, value);

diff --git a/config.c b/config.c
index e8ebef77d5c92423f7af85d9176059e2a772a320..589dec90282804431a4182da0a291543fe66ced3 100644 (file)
--- a/config.c
+++ b/config.c
@@ -362,7 +362,8 @@ static void populate_remote_urls(struct config_include_data *inc)
        current_parsing_scope = store_scope;
 }
 
-static int forbid_remote_url(const char *var, const char *value, void *data)
+static int forbid_remote_url(const char *var, const char *UNUSED(value),
+                            void *UNUSED(data))
 {
        const char *remote_name;
        size_t remote_name_len;

diff --git a/convert.c b/convert.c
index 4d153729da0f185207b85d103905b30e3183796b..b31a25b536041becdfce19e9b7d2f9673f86c8ef 100644 (file)
--- a/convert.c
+++ b/convert.c
@@ -1008,7 +1008,7 @@ static int apply_filter(const char *path, const char *src, size_t len,
        return 0;
 }
 
-static int read_convert_config(const char *var, const char *value, void *cb)
+static int read_convert_config(const char *var, const char *value, void *UNUSED(cb))
 {
        const char *key, *name;
        size_t namelen;

diff --git a/delta-islands.c b/delta-islands.c
index 13eb96e0c4b3770bad9d4c2e6049c9693545728f..c64333f9de2e56f985f9f6fc58ac7bebd227a93f 100644 (file)
--- a/delta-islands.c
+++ b/delta-islands.c
@@ -316,7 +316,7 @@ static regex_t *island_regexes;
 static unsigned int island_regexes_alloc, island_regexes_nr;
 static const char *core_island_name;
 
-static int island_config_callback(const char *k, const char *v, void *cb)
+static int island_config_callback(const char *k, const char *v, void *UNUSED(cb))
 {
        if (!strcmp(k, "pack.island")) {
                struct strbuf re = STRBUF_INIT;

diff --git a/diff.c b/diff.c
index 974626a6211514b25af43e8f48933c6929826aa5..8a9c9083f35cd186a55b83ab563d582c3d34a2c9 100644 (file)
--- a/diff.c
+++ b/diff.c
@@ -264,7 +264,8 @@ void init_diff_ui_defaults(void)
        diff_detect_rename_default = DIFF_DETECT_RENAME;
 }
 
-int git_diff_heuristic_config(const char *var, const char *value, void *cb)
+int git_diff_heuristic_config(const char *var, const char *value,
+                             void *UNUSED(cb))
 {
        if (!strcmp(var, "diff.indentheuristic"))
                diff_indent_heuristic = git_config_bool(var, value);

diff --git a/git-compat-util.h b/git-compat-util.h
index c6669db07d113ae969e4dfb1e7529269ff2e25b0..12239fedf7b425ade7c02351ed933e4a048bd5de 100644 (file)
--- a/git-compat-util.h
+++ b/git-compat-util.h
@@ -403,7 +403,9 @@ typedef uintmax_t timestamp_t;
 #endif
 
 #ifndef platform_core_config
-static inline int noop_core_config(const char *var, const char *value, void *cb)
+static inline int noop_core_config(const char *UNUSED(var),
+                                  const char *UNUSED(value),
+                                  void *UNUSED(cb))
 {
        return 0;
 }

diff --git a/gpg-interface.c b/gpg-interface.c
index 6dff2414603ef8275e5fcaf453e5712db240575c..721d69bf42a93cfd2e6197b14dc5c72462c9f65f 100644 (file)
--- a/gpg-interface.c
+++ b/gpg-interface.c
@@ -699,7 +699,7 @@ void set_signing_key(const char *key)
        configured_signing_key = xstrdup(key);
 }
 
-int git_gpg_config(const char *var, const char *value, void *cb)
+int git_gpg_config(const char *var, const char *value, void *UNUSED(cb))
 {
        struct gpg_format *fmt = NULL;
        char *fmtname = NULL;

diff --git a/ident.c b/ident.c
index 7f66beda428ced55235fa9507e68cfa90cb72209..48745a1f0e9041e0e1c01ae78739c881576b4d3a 100644 (file)
--- a/ident.c
+++ b/ident.c
@@ -668,7 +668,7 @@ static int set_ident(const char *var, const char *value)
        return 0;
 }
 
-int git_ident_config(const char *var, const char *value, void *data)
+int git_ident_config(const char *var, const char *value, void *UNUSED(data))
 {
        if (!strcmp(var, "user.useconfigonly")) {
                ident_use_config_only = git_config_bool(var, value);

diff --git a/ll-merge.c b/ll-merge.c
index 14b83620191019e465291d13be023df794ffb645..9f3ae1f8fe1d8cd044f644bfc6d35d46806dda49 100644 (file)
--- a/ll-merge.c
+++ b/ll-merge.c
@@ -249,7 +249,8 @@ static enum ll_merge_result ll_ext_merge(const struct ll_merge_driver *fn,
 static struct ll_merge_driver *ll_user_merge, **ll_user_merge_tail;
 static const char *default_ll_merge;
 
-static int read_merge_config(const char *var, const char *value, void *cb)
+static int read_merge_config(const char *var, const char *value,
+                            void *UNUSED(cb))
 {
        struct ll_merge_driver *fn;
        const char *key, *name;

diff --git a/ls-refs.c b/ls-refs.c
index 98e69373c8404ead982595fc9dbda5c4fe8137a1..e54b883e8790b7d999d138651fa261818dcaa0e5 100644 (file)
--- a/ls-refs.c
+++ b/ls-refs.c
@@ -136,7 +136,8 @@ static void send_possibly_unborn_head(struct ls_refs_data *data)
        strbuf_release(&namespaced);
 }
 
-static int ls_refs_config(const char *var, const char *value, void *data)
+static int ls_refs_config(const char *var, const char *value,
+                         void *UNUSED(data))
 {
        /*
         * We only serve fetches over v2 for now, so respect only "uploadpack"

diff --git a/pager.c b/pager.c
index 5cfe23b025c77b08715591c3893017111c2a697d..19c016ff54976c5474bed1b3404dda039a6b360c 100644 (file)
--- a/pager.c
+++ b/pager.c
@@ -38,7 +38,8 @@ static void wait_for_pager_signal(int signo)
        raise(signo);
 }
 
-static int core_pager_config(const char *var, const char *value, void *data)
+static int core_pager_config(const char *var, const char *value,
+                            void *UNUSED(data))
 {
        if (!strcmp(var, "core.pager"))
                return git_config_string(&pager_program, var, value);

diff --git a/pretty.c b/pretty.c
index 6d819103fbf1db3a201f993e20140bfa288a0e12..584026b746919805e21b69262942673e2036391f 100644 (file)
--- a/pretty.c
+++ b/pretty.c
@@ -43,7 +43,8 @@ static void save_user_format(struct rev_info *rev, const char *cp, int is_tforma
        rev->commit_format = CMIT_FMT_USERFORMAT;
 }
 
-static int git_pretty_formats_config(const char *var, const char *value, void *cb)
+static int git_pretty_formats_config(const char *var, const char *value,
+                                    void *UNUSED(cb))
 {
        struct cmt_fmt_map *commit_format = NULL;
        const char *name;

diff --git a/submodule.c b/submodule.c
index d99c978fa5b4cc80a888c7cbb226df4139eb0e72..597a347f859be7632d2f3f5d176635014d75160a 100644 (file)
--- a/submodule.c
+++ b/submodule.c
@@ -213,7 +213,8 @@ void set_diffopt_flags_from_submodule_config(struct diff_options *diffopt,
 }
 
 /* Cheap function that only determines if we're interested in submodules at all */
-int git_default_submodule_config(const char *var, const char *value, void *cb)
+int git_default_submodule_config(const char *var, const char *value,
+                                void *UNUSED(cb))
 {
        if (!strcmp(var, "submodule.recurse")) {
                int v = git_config_bool(var, value) ?

diff --git a/t/helper/test-config.c b/t/helper/test-config.c
index a6e936721fe660bb110e3d691d61dad80fddc307..ddd538b838d7eba4d0aaf12815437d9ffe3919fa 100644 (file)
--- 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 f013f8a31e5294d879f1e872c7f37626fa583ed6..64538a0c203dce44b8e58706f464becdff5f2ac7 100644 (file)
--- 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;

diff --git a/trailer.c b/trailer.c
index d419c20735e993d5812f396428e7727533d56768..a1e80478ab04c167c7e9337756430e6d1b4cd88e 100644 (file)
--- a/trailer.c
+++ b/trailer.c
@@ -478,7 +478,8 @@ static struct {
        { "ifmissing", TRAILER_IF_MISSING }
 };
 
-static int git_trailer_default_config(const char *conf_key, const char *value, void *cb)
+static int git_trailer_default_config(const char *conf_key, const char *value,
+                                     void *UNUSED(cb))
 {
        const char *trailer_item, *variable_name;
 
@@ -509,7 +510,8 @@ static int git_trailer_default_config(const char *conf_key, const char *value, v
        return 0;
 }
 
-static int git_trailer_config(const char *conf_key, const char *value, void *cb)
+static int git_trailer_config(const char *conf_key, const char *value,
+                             void *UNUSED(cb))
 {
        const char *trailer_item, *variable_name;
        struct arg_item *item;