[PATCH 3/5] sequencer: remove superfluous conditional
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Dec 22, 2017, 23:55 UTC
- Message-ID
- <2e30f42e0bbe6d1f86f46e6461d3218adb2ede5b.1513986836.git.johannes.schindelin@gmx.de>
- In-Reply-To
- <cover.1513986836.git.johannes.schindelin@gmx.de>
In a conditional block that is only reached when handling a TODO_REWORD (as seen even from a 3-line context), there is absolutely no need to nest another block under the identical condition.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> --- sequencer.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/sequencer.c b/sequencer.c index 8e6b6289be6..38266c3c228 100644 --- a/sequencer.c +++ b/sequencer.c @@ -1011,9 +1011,7 @@ static int do_pick_commit(enum todo_command command, struct commit *commit, opts); if (res || command != TODO_REWORD) goto leave; - flags |= EDIT_MSG | AMEND_MSG; - if (command == TODO_REWORD) - flags |= VERIFY_MSG; + flags |= EDIT_MSG | AMEND_MSG | VERIFY_MSG; msg_file = NULL; goto fast_forward_edit; }
-- 2.15.1.windows.2