diff options
author | René Scharfe <l.s.r@web.de> | 2022-12-13 09:27:10 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-12-13 16:07:08 +0300 |
commit | 4cb39fcf19b969cdfa042b65d4d0b62a7fd0ba1e (patch) | |
tree | 2f66f2fbfede97fe8883a2c168cf1e1c87713ae5 /commit.c | |
parent | 8706a59933d09354c5e3eb09a543453655a97183 (diff) |
commit: skip already cleared parents in clear_commit_marks_1()
Don't put clean parents on the pending list, as they and their ancestors
don't need any treatment and would be skipped later anyway. This saves
the allocation and release of a commit list item in ca. 20% of the cases
during a run of the test suite.
Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.c')
-rw-r--r-- | commit.c | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -693,8 +693,10 @@ static void clear_commit_marks_1(struct commit_list **plist, if (!parents) return; - while ((parents = parents->next)) - commit_list_insert(parents->item, plist); + while ((parents = parents->next)) { + if (parents->item->object.flags & mark) + commit_list_insert(parents->item, plist); + } commit = commit->parents->item; } |