Skip to content

Commit 5f8f927

Browse files
dschogitster
authored andcommitted
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 <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent aee42e1 commit 5f8f927

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

sequencer.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1013,9 +1013,7 @@ static int do_pick_commit(enum todo_command command, struct commit *commit,
10131013
opts);
10141014
if (res || command != TODO_REWORD)
10151015
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;
10191017
msg_file = NULL;
10201018
goto fast_forward_edit;
10211019
}

0 commit comments

Comments
 (0)