remove_all_fetch_refspecs(): drop unused "remote" parameter

This function already takes a "key" parameter which uniquely identifies
the config key that we need to remove. There's no need for it to look at
the "remote" parameter at all. Let's drop it in the name of simplicity.

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

diff --git a/builtin/remote.c b/builtin/remote.c
index f7edf7f2cb1f5880e5674f0bf5d41672602e52e4..5591cef775432d2bd01b866183e789952ad36915 100644 (file)
--- a/builtin/remote.c
+++ b/builtin/remote.c
@@ -1407,7 +1407,7 @@ static int update(int argc, const char **argv)
        return retval;
 }
 
        return retval;
 }
 

-static int remove_all_fetch_refspecs(const char *remote, const char *key)
+static int remove_all_fetch_refspecs(const char *key)

 {
        return git_config_set_multivar_gently(key, NULL, NULL, 1);
 }
 {
        return git_config_set_multivar_gently(key, NULL, NULL, 1);
 }


@@ -1437,7 +1437,7 @@ static int set_remote_branches(const char *remotename, const char **branches,
        if (!remote_is_configured(remote, 1))
                die(_("No such remote '%s'"), remotename);
 
        if (!remote_is_configured(remote, 1))
                die(_("No such remote '%s'"), remotename);
 

-       if (!add_mode && remove_all_fetch_refspecs(remotename, key.buf)) {
+       if (!add_mode && remove_all_fetch_refspecs(key.buf)) {

                strbuf_release(&key);
                return 1;
        }
                strbuf_release(&key);
                return 1;
        }