[PATCH STABLE] statprof: sort by time percentage as a secondary key

Yuya Nishihara yuya at tcha.org
Wed Oct 24 06:55:52 EDT 2018


On Tue, 23 Oct 2018 23:37:20 -0400, Matt Harbison wrote:
> # HG changeset patch
> # User Matt Harbison <matt_harbison at yahoo.com>
> # Date 1540351037 14400
> #      Tue Oct 23 23:17:17 2018 -0400
> # Branch stable
> # Node ID 3662c0ac3ab4b256ecabb348cc91605ee8973abb
> # Parent  49c7b701fdc2a8659b6f752aef507764d40ed5d0
> statprof: sort by time percentage as a secondary key
> 
> On Windows, os.times() only returns user and system times.  Real elapsed time is
> 0.  That results in no actual times reported, an end wall time of 0.000000, and
> seemingly randomly sorted stack frames.  This at least provides test stability
> in test-profile.t.

Sounds like we should change the default back to 'cpu' globally, or at least
on Windows.


More information about the Mercurial-devel mailing list