# HG changeset patch # User Martin von Zweigbergk # Date 1545240822 28800 # Node ID e8e2a7656e83ccb1444af8c13b7cc4b9d172bec5 # Parent fcc0a7ac9ebd0a1f97e593457470dd785f0b2151 help: hide default value for default-off flags If we no longer show the "[no-]" for default-off flags, it also seems unnecessary to show the "default: off" for them, since that's quite clearly the default. It's extra confusing for action flags like `hg bookmarks --delete`. Differential Revision: https://phab.mercurial-scm.org/D5455 diff -r fcc0a7ac9ebd -r e8e2a7656e83 mercurial/help.py --- a/mercurial/help.py Wed Dec 19 09:20:32 2018 -0800 +++ b/mercurial/help.py Wed Dec 19 09:33:42 2018 -0800 @@ -165,14 +165,14 @@ if isinstance(default, fancyopts.customopt): default = default.getdefaultvalue() - if (default and not callable(default)) or default is False: + if default and not callable(default): # default is of unknown type, and in Python 2 we abused # the %s-shows-repr property to handle integers etc. To # match that behavior on Python 3, we do str(default) and # then convert it to bytes. defaultstr = pycompat.bytestr(default) - if isinstance(default, bool): - defaultstr = _("on") if default else _("off") + if default is True: + defaultstr = _("on") desc += _(" (default: %s)") % defaultstr if isinstance(default, list): diff -r fcc0a7ac9ebd -r e8e2a7656e83 tests/test-extension.t --- a/tests/test-extension.t Wed Dec 19 09:20:32 2018 -0800 +++ b/tests/test-extension.t Wed Dec 19 09:33:42 2018 -0800 @@ -716,7 +716,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -755,7 +755,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -1043,7 +1043,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -1080,7 +1080,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -1155,7 +1155,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -1191,7 +1191,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) diff -r fcc0a7ac9ebd -r e8e2a7656e83 tests/test-fix.t --- a/tests/test-fix.t Wed Dec 19 09:20:32 2018 -0800 +++ b/tests/test-fix.t Wed Dec 19 09:33:42 2018 -0800 @@ -104,12 +104,12 @@ options ([+] can be repeated): - --all fix all non-public non-obsolete revisions (default: off) + --all fix all non-public non-obsolete revisions --base REV [+] revisions to diff against (overrides automatic selection, and applies to every revision being fixed) -r --rev REV [+] revisions to fix - -w --working-dir fix the working directory (default: off) - --whole always fix every line of a file (default: off) + -w --working-dir fix the working directory + --whole always fix every line of a file (some details hidden, use --verbose to show complete help) diff -r fcc0a7ac9ebd -r e8e2a7656e83 tests/test-help.t --- a/tests/test-help.t Wed Dec 19 09:20:32 2018 -0800 +++ b/tests/test-help.t Wed Dec 19 09:33:42 2018 -0800 @@ -440,7 +440,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -540,7 +540,7 @@ --profile print command execution profile --version output version information and exit -h --help display help and exit - --hidden consider hidden changesets (default: off) + --hidden consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -905,7 +905,7 @@ xxxxxxxxxxxxxxxxxxxxxxx (default: 3) -n -- normal desc --newline VALUE line1 line2 - --default-off enable X (default: off) + --default-off enable X --[no-]default-on enable Y (default: on) --callableopt VALUE adds foo --customopt VALUE adds bar @@ -2890,7 +2890,7 @@ display help and exit --hidden - consider hidden changesets (default: off) + consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) @@ -3094,7 +3094,7 @@ display help and exit --hidden - consider hidden changesets (default: off) + consider hidden changesets --pager TYPE when to paginate (boolean, always, auto, or never) (default: auto) diff -r fcc0a7ac9ebd -r e8e2a7656e83 tests/test-narrow-trackedcmd.t --- a/tests/test-narrow-trackedcmd.t Wed Dec 19 09:20:32 2018 -0800 +++ b/tests/test-narrow-trackedcmd.t Wed Dec 19 09:33:42 2018 -0800 @@ -105,9 +105,8 @@ --import-rules VALUE import narrowspecs from a file --removeexclude VALUE [+] old paths to no longer exclude --clear whether to replace the existing narrowspec - (default: off) --force-delete-local-changes forces deletion of local changes when - narrowing (default: off) + narrowing -e --ssh CMD specify ssh command to use --remotecmd CMD specify hg command to run on the remote side --insecure do not verify server certificate (ignoring diff -r fcc0a7ac9ebd -r e8e2a7656e83 tests/test-shelve.t --- a/tests/test-shelve.t Wed Dec 19 09:20:32 2018 -0800 +++ b/tests/test-shelve.t Wed Dec 19 09:33:42 2018 -0800 @@ -75,7 +75,7 @@ --cleanup delete all shelved changes --date DATE shelve with the specified commit date -d --delete delete the named shelved change(s) - -e --edit invoke editor on commit messages (default: off) + -e --edit invoke editor on commit messages -l --list list current shelves -m --message TEXT use text as shelve message -n --name NAME use the given name for the shelved commit diff -r fcc0a7ac9ebd -r e8e2a7656e83 tests/test-uncommit.t --- a/tests/test-uncommit.t Wed Dec 19 09:20:32 2018 -0800 +++ b/tests/test-uncommit.t Wed Dec 19 09:33:42 2018 -0800 @@ -34,8 +34,7 @@ options ([+] can be repeated): - --keep allow an empty commit after uncommiting (default: - off) + --keep allow an empty commit after uncommiting -I --include PATTERN [+] include names matching the given patterns -X --exclude PATTERN [+] exclude names matching the given patterns