[PATCH] revset: added _hexlist method to replace _list for %ln

Matt Mackall mpm at selenic.com
Thu Feb 27 12:17:40 CST 2014


On Wed, 2014-02-26 at 17:19 -0800, Lucas Moscovicz wrote:
> # HG changeset patch
> # User Lucas Moscovicz <lmoscovicz at fb.com>
> # Date 1393463755 28800
> #      Wed Feb 26 17:15:55 2014 -0800
> # Node ID 34bf12ea670796e0da95f44bf4ef95c1e0da8fd2
> # Parent  d9cf5e6843d7b15fe82ab0cd64a614489ff590da
> revset: added _hexlist method to replace _list for %ln
> 
> Now %ln expression goes through _hexlist and doesn't do any unnecesary
> processing anymore.

Queued for default, but you forgot to include your benchmarks.

You can still promote two scope lookups out of the inner loop.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list