]> git.ipfire.org Git - thirdparty/git.git/commitdiff
ref_transaction_commit(): inline call to write_ref_sha1()
authorMichael Haggerty <mhagger@alum.mit.edu>
Sat, 9 May 2015 15:29:20 +0000 (17:29 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 13 May 2015 04:28:03 +0000 (21:28 -0700)
That was the last caller, so delete function write_ref_sha1().

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
refs.c

diff --git a/refs.c b/refs.c
index ad4b6d520281579c9814c6b8cfdb7f238f645299..6cb1476c330cacf5bca481d2c1c53ed9b6a94487 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -3108,21 +3108,6 @@ static int commit_ref_update(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)
 {
@@ -3816,9 +3801,13 @@ int ref_transaction_commit(struct ref_transaction *transaction,
                                 */
                                unlock_ref(update->lock);
                                update->lock = NULL;
-                       } else if (write_ref_sha1(update->lock, update->new_sha1,
-                                                 update->msg)) {
-                               update->lock = NULL; /* freed by write_ref_sha1 */
+                       } else if (write_ref_to_lockfile(update->lock,
+                                                        update->new_sha1) ||
+                                  commit_ref_update(update->lock,
+                                                    update->new_sha1,
+                                                    update->msg)) {
+                               /* freed by one of the above calls: */
+                               update->lock = NULL;
                                strbuf_addf(err, "Cannot update the ref '%s'.",
                                            update->refname);
                                ret = TRANSACTION_GENERIC_ERROR;