Skip to content

Commit

Permalink
refs.c: make ref_transaction_create a wrapper for ref_transaction_update
Browse files Browse the repository at this point in the history
The ref_transaction_update function can already be used to create refs by
passing null_sha1 as the old_sha1 parameter. Simplify by replacing
transaction_create with a thin wrapper.

Signed-off-by: Ronnie Sahlberg <sahlberg@google.com>
Signed-off-by: Stefan Beller <sbeller@google.com>
Reviewed-by: Michael Haggerty <mhagger@alum.mit.edu>
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Ronnie Sahlberg authored and Junio C Hamano committed Dec 4, 2014
1 parent b260d26 commit bc9f292
Showing 1 changed file with 2 additions and 25 deletions.
27 changes: 2 additions & 25 deletions refs.c
Original file line number Diff line number Diff line change
Expand Up @@ -3623,31 +3623,8 @@ int ref_transaction_create(struct ref_transaction *transaction,
int flags, const char *msg,
struct strbuf *err)
{
struct ref_update *update;

assert(err);

if (transaction->state != REF_TRANSACTION_OPEN)
die("BUG: create called for transaction that is not open");

if (!new_sha1 || is_null_sha1(new_sha1))
die("BUG: create ref with null new_sha1");

if (check_refname_format(refname, REFNAME_ALLOW_ONELEVEL)) {
strbuf_addf(err, "refusing to create ref with bad name %s",
refname);
return -1;
}

update = add_update(transaction, refname);

hashcpy(update->new_sha1, new_sha1);
hashclr(update->old_sha1);
update->flags = flags;
update->have_old = 1;
if (msg)
update->msg = xstrdup(msg);
return 0;
return ref_transaction_update(transaction, refname, new_sha1,
null_sha1, flags, 1, msg, err);
}

int ref_transaction_delete(struct ref_transaction *transaction,
Expand Down

0 comments on commit bc9f292

Please sign in to comment.