On 08/03/18 17:53, Junio C Hamano wrote: > Phillip Wood <phillip.wood@xxxxxxxxxxxx> writes: > >> and use a leading '-' for inversion. I'm tempted to keep supporting 'n-' >> to mean everything from 'n' to the last line though. > > Thanks for double checking. It would be a better endgame to follow > up with an update to existing "range selection" code to also support > "n-", if you go that route. > I'm afraid I'm not sure exactly what you're suggesting. At the moment the range selection code is in the first patch and supports incomplete ranges. Are you suggesting that support for incomplete ranges should be in a separate patch or have I misunderstood? Thanks Phillip