[PATCH 6 of 6] revset: fix a crash with 'roots(wdir())'

Yuya Nishihara yuya at tcha.org
Wed Jul 1 08:01:17 CDT 2015


On Tue, 30 Jun 2015 23:31:32 -0400, Matt Harbison wrote:
> On Tue, 30 Jun 2015 23:24:32 -0400, Pierre-Yves David  
> <pierre-yves.david at ens-lyon.org> wrote:
> > On 06/30/2015 07:56 PM, Matt Harbison wrote:
> >> # HG changeset patch
> >> # User Matt Harbison <matt_harbison at yahoo.com>
> >> # Date 1435715342 14400
> >> #      Tue Jun 30 21:49:02 2015 -0400
> >> # Node ID c3aecbbe50596e54402e4bd98931d33f1b2d5d01
> >> # Parent  e26c2caf800020627f58e45f799ce932c6aee761
> >> revset: fix a crash with 'roots(wdir())'
> >
> > This series worries me about two things:
> >
> > 1) performance impact, can we get than benchmarked
> 
> I've seen you do benchmarking patches before, but admittedly didn't pay  
> much attention to what you were doing or how to benchmark.  Can you point  
> me to some specific steps to perform?
> 
> > 2) this is a lot of churn using None, could we settle for another value  
> > (max int) and move toward that instead?
> 
> https://www.selenic.com/pipermail/mercurial-devel/2015-June/071656.html

It means all "None"s in this series will have to be replaced by wdirrev.
Perhaps I should send my wdirrev patches soon.

I want to add one more thing:

3) can we have a function that is as fast as cl.parentrevs(r) and can process
   wdir() ?


More information about the Mercurial-devel mailing list