use pop_commit() for consuming the first entry of a struct commit_list

diff --git a/builtin/reflog.c b/builtin/reflog.c
index c2eb8ff8402068d7480a3afd262219cb03841ae1..fa841b500bcb006265c72b5f6a553b3ac24fa980 100644 (file)
--- a/builtin/reflog.c
+++ b/builtin/reflog.c
@@ -216,7 +216,6 @@ static int keep_entry(struct commit **it, unsigned char *sha1)
  */
 static void mark_reachable(struct expire_reflog_policy_cb *cb)
 {
-       struct commit *commit;
        struct commit_list *pending;
        unsigned long expire_limit = cb->mark_limit;
        struct commit_list *leftover = NULL;
@@ -226,11 +225,8 @@ static void mark_reachable(struct expire_reflog_policy_cb *cb)
 
        pending = cb->mark_list;
        while (pending) {
-               struct commit_list *entry = pending;
                struct commit_list *parent;
-               pending = entry->next;
-               commit = entry->item;
-               free(entry);
+               struct commit *commit = pop_commit(&pending);
                if (commit->object.flags & REACHABLE)
                        continue;
                if (parse_commit(commit))