sequencer: make rearrange_squash() a bit more obvious
authorJohannes Schindelin <johannes.schindelin@gmx.de>
Sun, 11 Feb 2018 00:10:08 +0000 (01:10 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 12 Feb 2018 22:43:36 +0000 (14:43 -0800)
There are some commands that have to be skipped from rearranging by virtue
of not handling any commits.

However, the logic was not quite obvious: it skipped commands based on
their position in the enum todo_command.

Instead, let's make it explicit that we skip all commands that do not
handle any commit. With one exception: the `drop` command, because it,
well, drops the commit and is therefore not eligible to rearranging.

Note: this is a bit academic at the moment because the only time we call
`rearrange_squash()` is directly after generating the todo list, when we
have nothing but `pick` commands anyway.

However, the upcoming `merge` command *will* want to be handled by that
function, and it *can* handle commits.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
sequencer.c
index 114db3b277577cc05770fa530a4f79311a28e7e6..764ad43388fb3e36cd61a9f004cc9858e933fe73 100644 (file)
@@ -2890,7 +2890,7 @@ int rearrange_squash(void)
                struct subject2item_entry *entry;
 
                next[i] = tail[i] = -1;
-               if (item->command >= TODO_EXEC) {
+               if (!item->commit || item->command == TODO_DROP) {
                        subjects[i] = NULL;
                        continue;
                }