diff options
| author | Jeff King <peff@peff.net> | 2016-11-04 12:30:12 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-11-09 14:56:21 -0800 |
| commit | 4bd488ea7c56b673947b07511d26f2afc4ec0bd6 (patch) | |
| tree | edb43e113c2aac7c150105de434e7f89c9905ae8 /builtin/checkout.c | |
| parent | 0202c411edc25940cc381bf317badcdf67670be4 (diff) | |
| download | git-4bd488ea7c56b673947b07511d26f2afc4ec0bd6.tar.gz | |
create_branch: drop unused "head" parameterjk/create-branch-remove-unused-param
This function used to have the caller pass in the current
value of HEAD, in order to make sure we didn't clobber HEAD.
In 55c4a6730, that logic moved to validate_new_branchname(),
which just resolves HEAD itself. The parameter to
create_branch is now unused.
Since we have to update and re-wrap the docstring describing
the parameters anyway, let's take this opportunity to break
it out into a list, which makes it easier to find the
parameters.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/checkout.c')
| -rw-r--r-- | builtin/checkout.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/checkout.c b/builtin/checkout.c index d3b296880e..d561f7491d 100644 --- a/builtin/checkout.c +++ b/builtin/checkout.c @@ -628,7 +628,7 @@ static void update_refs_for_switch(const struct checkout_opts *opts, } } else - create_branch(old->name, opts->new_branch, new->name, + create_branch(opts->new_branch, new->name, opts->new_branch_force ? 1 : 0, opts->new_branch_log, opts->new_branch_force ? 1 : 0, |