Re: Kidney blame's behaviour and edge cases

On 13.06.2013 10:35, Daniel Shahaf wrote:
> It seems to me it should ideally print '3' for every line, and the user
> should pass '-r 2:3' if he wants to distinguish "added in r3" from
> "added before r3". It would be easy to preserve the current behaviour,
> though, of printing '-' rather than '2' (where '2' here is the youngest
> change to that line, for lines added before r3).

It strikes me that what you're really looking for is either

    -rN:M -> [N, M-1],

which would make the two blame variants symmetrical, or

    -rN:M -> [N+1,M]

as you don't really have to find "the next interesting change" -- you
only have to make sure the first bound is inclusive, as with -c, rather
than exclusive, as with -r. That would also imply

     -cN:M -> [N+1,M]

and I'm not sure we currently do it that way.

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco // Non-Stop Data
e. brane_at_wandisco.com