]> git.ipfire.org Git - thirdparty/git.git/commit - sequencer.c
sequencer: refactor rearrange_squash() to work on a todo_list
authorAlban Gruin <alban.gruin@gmail.com>
Tue, 5 Mar 2019 19:17:55 +0000 (20:17 +0100)
committerJunio C Hamano <gitster@pobox.com>
Thu, 7 Mar 2019 00:17:57 +0000 (09:17 +0900)
commitf2a04904be6584f1ec783ed5d3c425026bcf908f
treed748ccafea452c7085dd686d1d3a8da0f5341c1c
parent683153a438f1b6b8e1a289a71f36244bde67e38f
sequencer: refactor rearrange_squash() to work on a todo_list

This refactors rearrange_squash() to work on a todo_list to avoid
redundant reads and writes.  The function is renamed
todo_list_rearrange_squash().

The old version created a new buffer, which was directly written to the
disk.  This new version creates a new item list by just copying items
from the old item list, without creating a new buffer.  This eliminates
the need to reparse the todo list, but this also means its buffer cannot
be directly written to the disk.

As rebase -p still need to check the todo list from the disk, a new
function is introduced, rearrange_squash_in_todo_file().

complete_action() still uses rearrange_squash_in_todo_file() for now.
This will be changed in a future commit.

Signed-off-by: Alban Gruin <alban.gruin@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/rebase--interactive.c
sequencer.c
sequencer.h