From: Phillip Wood <phillip.wood@talktalk.net>
Date: Tue, 13 Mar 2018 12:06:11 GMT
Subject: Re: [PATCH v2 2/3] add -p: allow line selection to be inverted
Message-ID: <ea4f645f-5cf2-ef84-24cf-fb585f96039d@talktalk.net>
In-Reply-To: <xmqq371axxm5.fsf@gitster-ct.c.googlers.com>

On 08/03/18 17:53, Junio C Hamano wrote:
> Phillip Wood <phillip.wood@talktalk.net> 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