Hi, On Mon, 28 Apr 2008, Brian Gernhardt wrote: > On Apr 28, 2008, at 9:46 AM, Johannes Schindelin wrote: > > >How about fixing the code to not need tac instead? We went to great > >lengths to introduce the --reverse option to the rev-list command, in > >order to avoid tac in the original version of rebase -i. > > Because this is reversing the output of a sub-shell (ll.395-471), not > rev-list. And making the shell code insert commands before the lines > the produce it would make the code complex at best. (And it's not > simple code either.) That's my _point_: the code is way too complex for what it tries to accomplish, namely implement a sane syntax for rebase -i -p. Ciao, Dscho -- 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
