diff options
| author | René Scharfe <l.s.r@web.de> | 2025-12-24 18:03:14 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2025-12-25 08:29:27 +0900 |
| commit | d8a17ef09b8d9fdeb7d22cbc926cbebf3d8a58c9 (patch) | |
| tree | 9d98efb22a494bdc5a9cf0c40c94365983eeb863 /revision.c | |
| parent | 66ce5f8e8872f0183bb137911c52b07f1f242d13 (diff) | |
| download | git-d8a17ef09b8d9fdeb7d22cbc926cbebf3d8a58c9.tar.xz | |
revision: export commit_stack
Dynamic arrays of commit pointers are used in several places. Some of
them use a custom struct to hold array, item count and capacity, others
have them as separate variables linked by a common name part.
Pick one succinct, clean implementation -- commit_stack -- and convert
the different variants to it to reduce code duplication.
Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'revision.c')
| -rw-r--r-- | revision.c | 23 |
1 files changed, 0 insertions, 23 deletions
diff --git a/revision.c b/revision.c index 5f0850ae5c..1858e093ee 100644 --- a/revision.c +++ b/revision.c @@ -250,29 +250,6 @@ void mark_trees_uninteresting_sparse(struct repository *r, paths_and_oids_clear(&map); } -struct commit_stack { - struct commit **items; - size_t nr, alloc; -}; -#define COMMIT_STACK_INIT { 0 } - -static void commit_stack_push(struct commit_stack *stack, struct commit *commit) -{ - ALLOC_GROW(stack->items, stack->nr + 1, stack->alloc); - stack->items[stack->nr++] = commit; -} - -static struct commit *commit_stack_pop(struct commit_stack *stack) -{ - return stack->nr ? stack->items[--stack->nr] : NULL; -} - -static void commit_stack_clear(struct commit_stack *stack) -{ - FREE_AND_NULL(stack->items); - stack->nr = stack->alloc = 0; -} - static void mark_one_parent_uninteresting(struct rev_info *revs, struct commit *commit, struct commit_stack *pending) { |
