From ad4cd6c29743274001cce323b670f7fb0c035ff1 Mon Sep 17 00:00:00 2001 From: Michael Haggerty Date: Sat, 9 May 2015 17:18:36 +0200 Subject: [PATCH] commit_ref_update(): new function, extracted from write_ref_sha1() Signed-off-by: Michael Haggerty Signed-off-by: Junio C Hamano --- refs.c | 27 ++++++++++++++++++++------- 1 file changed, 20 insertions(+), 7 deletions(-) diff --git a/refs.c b/refs.c index f94682a0b6..3db88e0b2a 100644 --- a/refs.c +++ b/refs.c @@ -3059,15 +3059,13 @@ static int write_ref_to_lockfile(struct ref_lock *lock, } /* - * Write sha1 into the ref specified by the lock. Make sure that errno - * is sane on error. + * Commit a change to a loose reference that has already been written + * to the loose reference lockfile. Also update the reflogs if + * necessary, using the specified lockmsg (which can be NULL). */ -static int write_ref_sha1(struct ref_lock *lock, - const unsigned char *sha1, const char *logmsg) +static int commit_ref_update(struct ref_lock *lock, + const unsigned char *sha1, const char *logmsg) { - if (write_ref_to_lockfile(lock, sha1)) - return -1; - clear_loose_ref_cache(&ref_cache); if (log_ref_write(lock->ref_name, lock->old_sha1, sha1, logmsg) < 0 || (strcmp(lock->ref_name, lock->orig_ref_name) && @@ -3106,6 +3104,21 @@ static int write_ref_sha1(struct ref_lock *lock, return 0; } +/* + * Write sha1 as the new value of the reference specified by the + * (open) lock. On error, roll back the lockfile and set errno + * appropriately. + */ +static int write_ref_sha1(struct ref_lock *lock, + const unsigned char *sha1, const char *logmsg) +{ + if (write_ref_to_lockfile(lock, sha1) || + commit_ref_update(lock, sha1, logmsg)) + return -1; + + return 0; +} + int create_symref(const char *ref_target, const char *refs_heads_master, const char *logmsg) { -- 2.11.4.GIT