commit-reach: simplify cleanup of remaining bitmaps in ahead_behind ()

Don't bother extracting the last few remaining prio_queue items in
order when we only want to free their associated bitmaps; just iterate
over the item array.

Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/commit-reach.c b/commit-reach.c
index 9604bbdcce2f35ffd947ec0e16fed4e5204edbca..d3a9b3ed6fe56147a3c33bdba34743cc212abe85 100644 (file)
--- a/commit-reach.c
+++ b/commit-reach.c
@@ -1117,10 +1117,8 @@ void ahead_behind(struct repository *r,
 
        /* STALE is used here, PARENT2 is used by insert_no_dup(). */
        repo_clear_commit_marks(r, PARENT2 | STALE);
-       while (prio_queue_peek(&queue)) {
-               struct commit *c = prio_queue_get(&queue);
-               free_bit_array(c);
-       }
+       for (size_t i = 0; i < queue.nr; i++)
+               free_bit_array(queue.array[i].data);
        clear_bit_arrays(&bit_arrays);
        clear_prio_queue(&queue);
 }