Branko Čibej wrote on Thu, Jun 13, 2013 at 10:54:47 +0200:
> 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
> 

(I assume you meant M<N throughout your email)

>     -rN:M -> [N, M-1],
> 
> which would make the two blame variants symmetrical, or
> 

It would make them symmetrical in that they both extend the range one
revision in the "towards older" direction.  I think that's not the
symmetry we need.  My kappa/iota example outlines the kind of symmetry
I'm expecting.

>     -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.

If you want to make the first bound inclusive, you need to extend the
range in the opposite direction, into [next-younger-interesting-revision, M].
(And then you run into the problem that in M:N you can give M=0, but
there's no equivalent for that in the N:M case.)

> That would also imply
> 
>      -cN:M -> [N+1,M]
> 
> and I'm not sure we currently do it that way.

'svn blame' does not take the '-c' option, only '-r'.

Reply via email to