Browse Source

refs.c: make ref_transaction_create a wrapper for ref_transaction_update

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>
maint
Ronnie Sahlberg 10 years ago committed by Junio C Hamano
parent
commit
bc9f2925fb
  1. 27
      refs.c

27
refs.c

@ -3623,31 +3623,8 @@ int ref_transaction_create(struct ref_transaction *transaction, @@ -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,

Loading…
Cancel
Save