Mercurial > hg
changeset 29782:97bfc2e5fba5
dispatch: set profiling.enabled when profiling is enabled
We do this for other global command arguments. We don't for --profile
for reasons that are unknown to me. Probably because nobody has needed
it.
An upcoming patch will introduce a new consumer of the profiling
code. It doesn't have access to command line arguments. So let's
set the config option during argument processing.
We also remove a check for "options['profile']" because it is now
redundant.
| author | Gregory Szorc <gregory.szorc@gmail.com> |
|---|---|
| date | Sun, 14 Aug 2016 16:35:58 -0700 |
| parents | 2654a0aac80d |
| children | 5d44197c208b |
| files | mercurial/dispatch.py |
| diffstat | 1 files changed, 5 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/dispatch.py Sun Aug 14 16:30:44 2016 -0700 +++ b/mercurial/dispatch.py Sun Aug 14 16:35:58 2016 -0700 @@ -819,6 +819,10 @@ for ui_ in uis: ui_.setconfig('ui', opt, val, '--' + opt) + if options['profile']: + for ui_ in uis: + ui_.setconfig('profiling', 'enabled', 'true', '--profile') + if options['traceback']: for ui_ in uis: ui_.setconfig('ui', 'traceback', 'on', '--traceback') @@ -904,7 +908,7 @@ except error.SignatureError: raise error.CommandError(cmd, _("invalid arguments")) - if options['profile'] or ui.configbool('profiling', 'enabled'): + if ui.configbool('profiling', 'enabled'): return profiling.profile(ui, checkargs) else: return checkargs()