Skip to content

Commit 60d8b5a

Browse files
rscharfegitster
authored andcommitted
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>
1 parent 7ff1e8d commit 60d8b5a

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

commit-reach.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1117,10 +1117,8 @@ void ahead_behind(struct repository *r,
11171117

11181118
/* STALE is used here, PARENT2 is used by insert_no_dup(). */
11191119
repo_clear_commit_marks(r, PARENT2 | STALE);
1120-
while (prio_queue_peek(&queue)) {
1121-
struct commit *c = prio_queue_get(&queue);
1122-
free_bit_array(c);
1123-
}
1120+
for (size_t i = 0; i < queue.nr; i++)
1121+
free_bit_array(queue.array[i].data);
11241122
clear_bit_arrays(&bit_arrays);
11251123
clear_prio_queue(&queue);
11261124
}

0 commit comments

Comments
 (0)