diff options
| author | Jay Soffian <jaysoffian@gmail.com> | 2009-02-03 17:51:12 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2009-02-04 05:12:23 (GMT) |
| commit | e02f1762b257c50124fc528e8b60bf16e8bb7acf (patch) | |
| tree | f0545dac97f52abf8a4b175cfb15321bb92f5981 /builtin-remote.c | |
| parent | bc395643b606ffe7a5c01e2772ae0644c840f368 (diff) | |
| download | git-e02f1762b257c50124fc528e8b60bf16e8bb7acf.zip git-e02f1762b257c50124fc528e8b60bf16e8bb7acf.tar.gz git-e02f1762b257c50124fc528e8b60bf16e8bb7acf.tar.bz2 | |
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.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/builtin-remote.c b/builtin-remote.c index 5af4e64..9ccc8a9 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, |