struct ref_lock: delete the force_write member

Instead, compute the value when it is needed.

Signed-off-by: Stefan Beller <sbeller@google.com>
Edited-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Reviewed-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/refs.c b/refs.c
index b6a2535fd07425a391e278e96051b4ed870c4f51..5d8d57d3490a045e23e511088af8f789d7b537da 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -12,7 +12,6 @@ struct ref_lock {
        struct lock_file *lk;
        unsigned char old_sha1[20];
        int lock_fd;
        struct lock_file *lk;
        unsigned char old_sha1[20];
        int lock_fd;

-       int force_write;

 };
 
 /*
 };
 
 /*


@@ -2319,8 +2318,6 @@ static struct ref_lock *lock_ref_sha1_basic(const char *refname,
        lock->ref_name = xstrdup(refname);
        lock->orig_ref_name = xstrdup(orig_refname);
        ref_file = git_path("%s", refname);
        lock->ref_name = xstrdup(refname);
        lock->orig_ref_name = xstrdup(orig_refname);
        ref_file = git_path("%s", refname);

-       if ((flags & REF_NODEREF) && (type & REF_ISSYMREF))
-               lock->force_write = 1;

 
  retry:
        switch (safe_create_leading_directories(ref_file)) {
 
  retry:
        switch (safe_create_leading_directories(ref_file)) {


@@ -3788,8 +3785,15 @@ int ref_transaction_commit(struct ref_transaction *transaction,
                struct ref_update *update = updates[i];
 
                if (!is_null_sha1(update->new_sha1)) {
                struct ref_update *update = updates[i];
 
                if (!is_null_sha1(update->new_sha1)) {

-                       if (!update->lock->force_write &&
-                           !hashcmp(update->lock->old_sha1, update->new_sha1)) {
+                       int overwriting_symref = ((update->type & REF_ISSYMREF) &&
+                                                 (update->flags & REF_NODEREF));
+
+                       if (!overwriting_symref
+                           && !hashcmp(update->lock->old_sha1, update->new_sha1)) {
+                               /*
+                                * The reference already has the desired
+                                * value, so we don't need to write it.
+                                */

                                unlock_ref(update->lock);
                                update->lock = NULL;
                        } else if (write_ref_sha1(update->lock, update->new_sha1,
                                unlock_ref(update->lock);
                                update->lock = NULL;
                        } else if (write_ref_sha1(update->lock, update->new_sha1,