[MERGE] special case the no-args version of revision-info

John Arbash Meinel john at arbash-meinel.com
Thu Mar 13 21:15:36 GMT 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Michael Hudson wrote:
> Reading this post:
> 
> http://blog.orebokech.com/2008/03/emacs-in-bzr-initial-impressions.html
> 
> I was prompted to look at the implementation of version-info.  It seems
> to be a bit whole-history happy: RevisionSpec.in_branch calls
> revsion_history() and the command implementation calls
> get_revid_to_revno_map().
> 
> This bundle just makes it really fast for the no-arg case.  Perhaps it
> would be better to improve the interpretation of "revno:-$SMALLINT".
> 
> Cheers,
> mwh
> 

BB:approve

I think there was a patch a while ago that did better for the simple
integer case. Basically walking back from tip rather than walking the
full ancestry. I certainly think that using
'branch.last_revision_info()' is a decent thing.

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFH2Zl4JdeBCYSNAAMRAto6AJ0RrFZ8Mh/3G3CkjjZYMKeBopnXswCeIdUS
YP3Vva21yqaQhWRuOsBjhQo=
=tPqq
-----END PGP SIGNATURE-----



More information about the bazaar mailing list