summaryrefslogtreecommitdiff
path: root/builtin-remote.c
diff options
context:
space:
mode:
authorJay Soffian <jaysoffian@gmail.com>2009-02-03 12:51:12 -0500
committerJunio C Hamano <gitster@pobox.com>2009-02-03 21:12:23 -0800
commite02f1762b257c50124fc528e8b60bf16e8bb7acf (patch)
treef0545dac97f52abf8a4b175cfb15321bb92f5981 /builtin-remote.c
parentbc395643b606ffe7a5c01e2772ae0644c840f368 (diff)
downloadgit-e02f1762b257c50124fc528e8b60bf16e8bb7acf.tar.gz
builtin-remote: make rm() use properly named variable to hold return value
"i" is a loop counter and should not be used to hold a return value; use "result" instead which is consistent with the rest of builtin-remote.c. Signed-off-by: Jay Soffian <jaysoffian@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-remote.c')
-rw-r--r--builtin-remote.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/builtin-remote.c b/builtin-remote.c
index 5af4e643eb..9ccc8a90e9 100644
--- a/builtin-remote.c
+++ b/builtin-remote.c
@@ -356,7 +356,7 @@ static int rm(int argc, const char **argv)
struct known_remotes known_remotes = { NULL, NULL };
struct string_list branches = { NULL, 0, 0, 1 };
struct branches_for_remote cb_data = { NULL, &branches, &known_remotes };
- int i;
+ int i, result;
if (argc != 2)
usage_with_options(builtin_remote_usage, options);
@@ -397,14 +397,14 @@ static int rm(int argc, const char **argv)
* refs, which are invalidated when deleting a branch.
*/
cb_data.remote = remote;
- i = for_each_ref(add_branch_for_removal, &cb_data);
+ result = for_each_ref(add_branch_for_removal, &cb_data);
strbuf_release(&buf);
- if (!i)
- i = remove_branches(&branches);
+ if (!result)
+ result = remove_branches(&branches);
string_list_clear(&branches, 1);
- return i;
+ return result;
}
static void show_list(const char *title, struct string_list *list,