special-case {true,false,test} --help

This also removes their help texts.

function                                             old     new   delta
run_applet_no_and_exit                               442     452     +10
packed_usage                                       30713   30625     -88

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2015-04-21 16:00:41 +02:00
parent 7d86384b24
commit de5edadee2
4 changed files with 21 additions and 28 deletions

View File

@ -10,11 +10,9 @@
/* BB_AUDIT SUSv3 compliant */ /* BB_AUDIT SUSv3 compliant */
/* http://www.opengroup.org/onlinepubs/000095399/utilities/false.html */ /* http://www.opengroup.org/onlinepubs/000095399/utilities/false.html */
//usage:#define false_trivial_usage /* "false --help" is special-cased to ignore --help */
//usage: "" //usage:#define false_trivial_usage NOUSAGE_STR
//usage:#define false_full_usage "\n\n" //usage:#define false_full_usage ""
//usage: "Return an exit code of FALSE (1)"
//usage:
//usage:#define false_example_usage //usage:#define false_example_usage
//usage: "$ false\n" //usage: "$ false\n"
//usage: "$ echo $?\n" //usage: "$ echo $?\n"

View File

@ -39,14 +39,9 @@
//config: help //config: help
//config: Enable 64-bit support in test. //config: Enable 64-bit support in test.
/* "test --help" does not print help (POSIX compat), only "[ --help" does. /* "test --help" is special-cased to ignore --help */
* We display "<applet> EXPRESSION ]" here (not "<applet> EXPRESSION") //usage:#define test_trivial_usage NOUSAGE_STR
* Unfortunately, it screws up generated BusyBox.html. TODO. */ //usage:#define test_full_usage ""
//usage:#define test_trivial_usage
//usage: "EXPRESSION ]"
//usage:#define test_full_usage "\n\n"
//usage: "Check file types, compare values etc. Return a 0/1 exit code\n"
//usage: "depending on logical value of EXPRESSION"
//usage: //usage:
//usage:#define test_example_usage //usage:#define test_example_usage
//usage: "$ test 1 -eq 2\n" //usage: "$ test 1 -eq 2\n"

View File

@ -10,11 +10,9 @@
/* BB_AUDIT SUSv3 compliant */ /* BB_AUDIT SUSv3 compliant */
/* http://www.opengroup.org/onlinepubs/007904975/utilities/true.html */ /* http://www.opengroup.org/onlinepubs/007904975/utilities/true.html */
//usage:#define true_trivial_usage /* "true --help" is special-cased to ignore --help */
//usage: "" //usage:#define true_trivial_usage NOUSAGE_STR
//usage:#define true_full_usage "\n\n" //usage:#define true_full_usage ""
//usage: "Return an exit code of TRUE (0)"
//usage:
//usage:#define true_example_usage //usage:#define true_example_usage
//usage: "$ true\n" //usage: "$ true\n"
//usage: "$ echo $?\n" //usage: "$ echo $?\n"

View File

@ -748,23 +748,25 @@ void FAST_FUNC run_applet_no_and_exit(int applet_no, char **argv)
xfunc_error_retval = EXIT_FAILURE; xfunc_error_retval = EXIT_FAILURE;
applet_name = APPLET_NAME(applet_no); applet_name = APPLET_NAME(applet_no);
#if defined APPLET_NO_test
/* Special case. POSIX says "test --help" /* Special case. POSIX says "test --help"
* should be no different from e.g. "test --foo". * should be no different from e.g. "test --foo".
* Thus for "test", we skip --help check. * Thus for "test", we skip --help check.
* "true" and "false" are also special.
*/ */
if (applet_no != APPLET_NO_test) if (1
#if defined APPLET_NO_test
&& applet_no != APPLET_NO_test
#endif
#if defined APPLET_NO_true
&& applet_no != APPLET_NO_true
#endif #endif
{
if (argc == 2 && strcmp(argv[1], "--help") == 0) {
#if defined APPLET_NO_false #if defined APPLET_NO_false
/* Someone insisted that "false --help" must exit 1. Sigh */ && applet_no != APPLET_NO_false
if (applet_no != APPLET_NO_false)
#endif #endif
{ ) {
if (argc == 2 && strcmp(argv[1], "--help") == 0) {
/* Make "foo --help" exit with 0: */ /* Make "foo --help" exit with 0: */
xfunc_error_retval = 0; xfunc_error_retval = 0;
}
bb_show_usage(); bb_show_usage();
} }
} }