Taylor Blau <me@xxxxxxxxxxxx> writes: > On Wed, Aug 12, 2020 at 03:45:47PM +0200, René Scharfe wrote: >> The only caller of reschedule_last_action was removed by ef64bb328df >> (rebase: strip unused code in git-rebase--preserve-merges.sh, >> 2018-05-28); remove this unused shell function as well. > > Easy peasy ;-). Obviously correct, and thanks for cleaning this up. > > Reviewed-by: Taylor Blau <me@xxxxxxxxxxxx> > >> Signed-off-by: René Scharfe <l.s.r@xxxxxx> I thought preserve-merges backend is on its way out, but I do not have objection to have some part of it retire earlier than the remainder of it ;-) Will queue. Thanks. >> --- >> git-rebase--preserve-merges.sh | 10 ---------- >> 1 file changed, 10 deletions(-) >> >> diff --git a/git-rebase--preserve-merges.sh b/git-rebase--preserve-merges.sh >> index dec90e9af67..b9c71d2a71b 100644 >> --- a/git-rebase--preserve-merges.sh >> +++ b/git-rebase--preserve-merges.sh >> @@ -193,16 +193,6 @@ mark_action_done () { >> fi >> } >> >> -# Put the last action marked done at the beginning of the todo list >> -# again. If there has not been an action marked done yet, leave the list of >> -# items on the todo list unchanged. >> -reschedule_last_action () { >> - tail -n 1 "$done" | cat - "$todo" >"$todo".new >> - sed -e \$d <"$done" >"$done".new >> - mv -f "$todo".new "$todo" >> - mv -f "$done".new "$done" >> -} >> - >> append_todo_help () { >> gettext " >> Commands: >> -- >> 2.28.0 > > Thanks, > Taylor