[PATCH] profile: disable nested report in lsprof by default

Augie Fackler raf at durin42.com
Tue May 26 10:14:59 CDT 2015


On Tue, May 26, 2015 at 08:04:59AM -0500, Matt Mackall wrote:
> # HG changeset patch
> # User Matt Mackall <mpm at selenic.com>
> # Date 1432314863 18000
> #      Fri May 22 12:14:23 2015 -0500
> # Node ID 7616c2158b545fa74e5aa0b1d71ac2703417eba1
> # Parent  df1801b693ae6647631ba1876608bd0a3d7fd879
> profile: disable nested report in lsprof by default

Sure, seems better.

>
> The nesting makes profiles harder to read and often pushes important
> data off the end of the report.
>
> diff -r df1801b693ae -r 7616c2158b54 mercurial/dispatch.py
> --- a/mercurial/dispatch.py	Tue May 26 06:45:18 2015 -0500
> +++ b/mercurial/dispatch.py	Fri May 22 12:14:23 2015 -0500
> @@ -898,7 +898,7 @@
>      format = ui.config('profiling', 'format', default='text')
>      field = ui.config('profiling', 'sort', default='inlinetime')
>      limit = ui.configint('profiling', 'limit', default=30)
> -    climit = ui.configint('profiling', 'nested', default=5)
> +    climit = ui.configint('profiling', 'nested', default=0)
>
>      if format not in ['text', 'kcachegrind']:
>          ui.warn(_("unrecognized profiling format '%s'"
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> https://selenic.com/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list