diff options
author | Daan De Meyer <daan.j.demeyer@gmail.com> | 2021-01-30 14:13:04 +0000 |
---|---|---|
committer | Yu Watanabe <watanabe.yu+github@gmail.com> | 2021-01-31 13:14:02 +0900 |
commit | bc556335b1c568c98688cc1f586b5f753fcddac6 (patch) | |
tree | 2f9336fb6ea885d672e8c0e23300a448a554ce02 /src/systemctl/systemctl-compat-halt.c | |
parent | 16455ee2b1853238cf0e936735ff92e690a20253 (diff) | |
download | systemd-bc556335b1c568c98688cc1f586b5f753fcddac6.tar.gz |
tree-wide: Drop custom formatting for print() help messages
I think this formatting was originally used because it simplified
adding new options to the help messages. However, these days, most
tools their help message end with "\nSee the %s for details.\n" so
the final line almost never has to be edited which eliminates the
benefit of the custom formatting used for printf() help messages.
Let's make things more consistent and use the same formatting for
printf() help messages that we use everywhere else.
Prompted by https://github.com/systemd/systemd/pull/18355#discussion_r567241580
Diffstat (limited to 'src/systemctl/systemctl-compat-halt.c')
-rw-r--r-- | src/systemctl/systemctl-compat-halt.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/src/systemctl/systemctl-compat-halt.c b/src/systemctl/systemctl-compat-halt.c index 8e41bd6cb8..82cb3df6cd 100644 --- a/src/systemctl/systemctl-compat-halt.c +++ b/src/systemctl/systemctl-compat-halt.c @@ -36,16 +36,15 @@ static int halt_help(void) { " -w --wtmp-only Don't halt/power-off/reboot, just write wtmp record\n" " -d --no-wtmp Don't write wtmp record\n" " --no-wall Don't send wall message before halt/power-off/reboot\n" - "\nSee the %s for details.\n" - , program_invocation_short_name - , arg_action == ACTION_REBOOT ? " [ARG]" : "" - , ansi_highlight() - , arg_action == ACTION_REBOOT ? "Reboot" : - arg_action == ACTION_POWEROFF ? "Power off" : - "Halt" - , ansi_normal() - , link - ); + "\nSee the %s for details.\n", + program_invocation_short_name, + arg_action == ACTION_REBOOT ? " [ARG]" : "", + ansi_highlight(), + arg_action == ACTION_REBOOT ? "Reboot" : + arg_action == ACTION_POWEROFF ? "Power off" : + "Halt", + ansi_normal(), + link); return 0; } |