On 21 July 2017 at 00:27, Junio C Hamano <gitster@xxxxxxxxx> wrote: > I tend to agree with you that 1-3/10 may be better off being a > single patch (or 3/10 dropped, as Brandon is working on losing it > nearby). I would have expected 7-8/10 to be a single patch, as by > the time a reader reaches 07/10, because of the groundwork laid by > 04-06/10, it is obvious that the general direction is to allow the > caller, i.e. cmd_tag(), to make a call to setup_auto_pager() only in > some but not all circumstances, and 07/10 being faithful to the > original behaviour (only to be updated in 08/10) is somewhat counter > intuitive. It is not wrong per-se; it was just unexpected. Thanks for your comments. I will be away for a few days, but once I get back, I'll try to produce a v3 based on this and any further feedback. Martin
