[PATCH 1 of 6] perf: add a command to test addremove performance

Matt Mackall mpm at selenic.com
Wed Apr 3 14:48:16 CDT 2013


On Tue, 2013-04-02 at 16:06 -0700, Siddharth Agarwal wrote:
> On 04/02/2013 04:04 PM, Bryan O'Sullivan wrote:
> > This seems a bit wrong to me. The perf framework mostly tests internal 
> > chunks of code that are difficult to get at from the command line 
> > interface: addremove is not such a chunk of code, and can easily be 
> > measured with "hg --time".
> 
> Hmm, I thought the main point of perf was to actually do timing 
> measurements properly. perfstatus doesn't need to exist either since 
> it's easy enough to replicate with hg status -mard.

I think this is fine.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list