Mercurial > hg
changeset 31493:7e9c7d1d65cb
dispatch: extract maybe-use-repr formatting to helper function
I think this makes the code much clearer. I had to think for a bit to
unpack the old-school `condition and if-true or if-false` dance, and
formatting argument lists here shouldn't be performance critical.
| author | Augie Fackler <augie@google.com> |
|---|---|
| date | Sun, 19 Mar 2017 00:21:26 -0400 |
| parents | 3c77414a0f9c |
| children | faf75a701aca |
| files | mercurial/dispatch.py |
| diffstat | 1 files changed, 6 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/dispatch.py Sun Mar 19 00:18:53 2017 -0400 +++ b/mercurial/dispatch.py Sun Mar 19 00:21:26 2017 -0400 @@ -92,8 +92,13 @@ if inst.hint: write(_("(%s)\n") % inst.hint) +def _mayberepr(a): + if ' ' in a: + return repr(a) + return a + def _formatargs(args): - return ' '.join(' ' in a and repr(a) or a for a in args) + return ' '.join(_mayberepr(a) for a in args) def dispatch(req): "run the command specified in req.args"