sequencer: remove superfluous conditional
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> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Johannes Schindelin committed
Dec 23, 2017 at 00:55 UTC
5f8f927710a039ce3068ac55ca87354477a199b9
1 file changed
+1
-3
sequencer.c
+1
-3
@@ -1013,9 +1013,7 @@ static int do_pick_commit(enum todo_command command, struct commit *commit,
1013
opts);
1014
if (res || command != TODO_REWORD)
1015
goto leave;
1016
- flags |= EDIT_MSG | AMEND_MSG;
1017
- if (command == TODO_REWORD)
1018
- flags |= VERIFY_MSG;
1016
+ flags |= EDIT_MSG | AMEND_MSG | VERIFY_MSG;
1017
msg_file = NULL;
1018
goto fast_forward_edit;
1019
}