[PATCH 04/24] revision.[ch]: split freeing of revs->commit into a function
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Mar 9, 2022, 13:16 UTC
- Message-ID
- <patch-04.24-763e0ab3425-20220309T123321Z-avarab@gmail.com>
- In-Reply-To
- <cover-00.24-00000000000-20220309T123321Z-avarab@gmail.com>
Move the existing code for invoking free_commit_list() and setting revs->commits to NULL into a new release_revisions_commit_list() function. This will be used as part of a general free()-ing mechanism for "struct rev_info".
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- revision.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/revision.c b/revision.c index 2bb913f2f3f..5824fdeddfd 100644 --- a/revision.c +++ b/revision.c @@ -2916,6 +2916,16 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, struct s return left; } +static void release_revisions_commit_list(struct rev_info *revs) +{ + struct commit_list *commits = revs->commits; + + if (!commits) + return; + free_commit_list(commits); + revs->commits = NULL; +} + static void add_child(struct rev_info *revs, struct commit *parent, struct commit *child) { struct commit_list *l = xcalloc(1, sizeof(*l)); @@ -4073,10 +4083,7 @@ static void create_boundary_commit_list(struct rev_info *revs) * boundary commits anyway. (This is what the code has always * done.) */ - if (revs->commits) { - free_commit_list(revs->commits); - revs->commits = NULL; - } + release_revisions_commit_list(revs); /* * Put all of the actual boundary commits from revs->boundary_commits
-- 2.35.1.1295.g6b025d3e231