Junio C Hamano <gitster@xxxxxxxxx> writes: > The structure of this new test piece, including the use of "log -1 > --format", seems to follow existing tests, and very readable. Why > do we have one extra level of subshell, though? There is no "cd" > that may affect the later test pieces, and set_something_editor that > touches environment that may affect the later test pieces is called > in its own subshell already. > > Other than that, looking good (there may be a valid reason why the > test piece needs the subshell around it, but it was just not apparent > to me). Ah, now I notice that Phillip also noticed the same thing. I just removed the outer subshell while queuing. Thanks for working on this, and thanks Phillip for excellent reviews.