[PATCH] -r 1-100 not -r1:100 please...

Michael Ellerman michael at ellerman.id.au
Fri May 27 07:38:29 BST 2005


On Fri, 27 May 2005 15:04, John Arbash Meinel wrote:
> Rusty Russell wrote:
> >OK, you knew this was coming 8)
>
> Check out my latest patch for revision handling. There is a very
> specific reason to use ':' instead of '-'. Negative numbers.
>
> ...
>
> It isn't the end-all be-all, but I do think negative revisions numbers
> are a nice thing.

Yeah I think negative revision numbers are kind of "neat", but I don't think 
you want to design your UI around them.

By using - for ranges you free up : for seperating the name space prefix so as 
I see it, it's either:

bzr diff -r 10:20
and
bzr diff -r {date:yesterday}

OR

bzr diff -r 10-20
and
bzr diff -r date:yesterday

I think the second option is better because a) 10-20 is a much more common 
syntax for specifying a range (eg. print something), and getting rid of the 
{} is nice.

cheers


-- 
Michael Ellerman
IBM OzLabs

email: michael:ellerman.id.au
inmsg: mpe:jabber.org
wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050527/8ffc093e/attachment.pgp 


More information about the bazaar mailing list