builtin/branch: convert to struct object_id

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
brian m. carlson 2017-02-21 23:47:26 +00:00 committed by Junio C Hamano
parent 1db1108654
commit 0c77cd24f8

View File

@ -32,7 +32,7 @@ static const char * const builtin_branch_usage[] = {
}; };
static const char *head; static const char *head;
static unsigned char head_sha1[20]; static struct object_id head_oid;
static int branch_use_color = -1; static int branch_use_color = -1;
static char branch_colors[][COLOR_MAXLEN] = { static char branch_colors[][COLOR_MAXLEN] = {
@ -117,13 +117,13 @@ static int branch_merged(int kind, const char *name,
if (kind == FILTER_REFS_BRANCHES) { if (kind == FILTER_REFS_BRANCHES) {
struct branch *branch = branch_get(name); struct branch *branch = branch_get(name);
const char *upstream = branch_get_upstream(branch, NULL); const char *upstream = branch_get_upstream(branch, NULL);
unsigned char sha1[20]; struct object_id oid;
if (upstream && if (upstream &&
(reference_name = reference_name_to_free = (reference_name = reference_name_to_free =
resolve_refdup(upstream, RESOLVE_REF_READING, resolve_refdup(upstream, RESOLVE_REF_READING,
sha1, NULL)) != NULL) oid.hash, NULL)) != NULL)
reference_rev = lookup_commit_reference(sha1); reference_rev = lookup_commit_reference(oid.hash);
} }
if (!reference_rev) if (!reference_rev)
reference_rev = head_rev; reference_rev = head_rev;
@ -153,10 +153,10 @@ static int branch_merged(int kind, const char *name,
} }
static int check_branch_commit(const char *branchname, const char *refname, static int check_branch_commit(const char *branchname, const char *refname,
const unsigned char *sha1, struct commit *head_rev, const struct object_id *oid, struct commit *head_rev,
int kinds, int force) int kinds, int force)
{ {
struct commit *rev = lookup_commit_reference(sha1); struct commit *rev = lookup_commit_reference(oid->hash);
if (!rev) { if (!rev) {
error(_("Couldn't look up commit object for '%s'"), refname); error(_("Couldn't look up commit object for '%s'"), refname);
return -1; return -1;
@ -183,7 +183,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
int quiet) int quiet)
{ {
struct commit *head_rev = NULL; struct commit *head_rev = NULL;
unsigned char sha1[20]; struct object_id oid;
char *name = NULL; char *name = NULL;
const char *fmt; const char *fmt;
int i; int i;
@ -207,7 +207,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
} }
if (!force) { if (!force) {
head_rev = lookup_commit_reference(head_sha1); head_rev = lookup_commit_reference(head_oid.hash);
if (!head_rev) if (!head_rev)
die(_("Couldn't look up commit object for HEAD")); die(_("Couldn't look up commit object for HEAD"));
} }
@ -235,7 +235,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
RESOLVE_REF_READING RESOLVE_REF_READING
| RESOLVE_REF_NO_RECURSE | RESOLVE_REF_NO_RECURSE
| RESOLVE_REF_ALLOW_BAD_NAME, | RESOLVE_REF_ALLOW_BAD_NAME,
sha1, &flags); oid.hash, &flags);
if (!target) { if (!target) {
error(remote_branch error(remote_branch
? _("remote-tracking branch '%s' not found.") ? _("remote-tracking branch '%s' not found.")
@ -245,13 +245,13 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
} }
if (!(flags & (REF_ISSYMREF|REF_ISBROKEN)) && if (!(flags & (REF_ISSYMREF|REF_ISBROKEN)) &&
check_branch_commit(bname.buf, name, sha1, head_rev, kinds, check_branch_commit(bname.buf, name, &oid, head_rev, kinds,
force)) { force)) {
ret = 1; ret = 1;
goto next; goto next;
} }
if (delete_ref(name, is_null_sha1(sha1) ? NULL : sha1, if (delete_ref(name, is_null_oid(&oid) ? NULL : oid.hash,
REF_NODEREF)) { REF_NODEREF)) {
error(remote_branch error(remote_branch
? _("Error deleting remote-tracking branch '%s'") ? _("Error deleting remote-tracking branch '%s'")
@ -267,7 +267,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds,
bname.buf, bname.buf,
(flags & REF_ISBROKEN) ? "broken" (flags & REF_ISBROKEN) ? "broken"
: (flags & REF_ISSYMREF) ? target : (flags & REF_ISSYMREF) ? target
: find_unique_abbrev(sha1, DEFAULT_ABBREV)); : find_unique_abbrev(oid.hash, DEFAULT_ABBREV));
} }
delete_branch_config(bname.buf); delete_branch_config(bname.buf);
@ -693,7 +693,7 @@ int cmd_branch(int argc, const char **argv, const char *prefix)
track = git_branch_track; track = git_branch_track;
head = resolve_refdup("HEAD", 0, head_sha1, NULL); head = resolve_refdup("HEAD", 0, head_oid.hash, NULL);
if (!head) if (!head)
die(_("Failed to resolve HEAD as a valid ref.")); die(_("Failed to resolve HEAD as a valid ref."));
if (!strcmp(head, "HEAD")) if (!strcmp(head, "HEAD"))