Martin von Zweigbergk <martin.von.zweigbergk@xxxxxxxxx> writes: > On Thu, Aug 9, 2012 at 9:05 AM, Michael J Gruber > <git@xxxxxxxxxxxxxxxxxxxx> wrote: >> diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh >> index 0d2056f..dbc9de6 100644 >> --- a/git-rebase--interactive.sh >> +++ b/git-rebase--interactive.sh >> @@ -573,7 +573,7 @@ do_next () { >> newhead=$(git rev-parse HEAD) && >> case $head_name in >> refs/*) >> - message="$GIT_REFLOG_ACTION: $head_name onto $shortonto" && >> + message="$GIT_REFLOG_ACTION: $head_name onto $onto" && > > After this patch, is there any point in the > > shortonto=$(git rev-parse --short $onto) && > > line just before the context? (I can't see any.) Good point; the function always exits after reaching this point, so it should be safe to omit it. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html