On Wed, 5 Jun 2019, Daniel Wagner wrote:
> Add optstring to getopt_long() command line parser to support the
> short options as it documented in the man page and also in the usage
> help text.
>
> Signed-off-by: Daniel Wagner <wagi@xxxxxxxxx>
> ---
> src/pi_tests/pi_stress.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/pi_tests/pi_stress.c b/src/pi_tests/pi_stress.c
> index ac7646cd2613..543106be8e18 100644
> --- a/src/pi_tests/pi_stress.c
> +++ b/src/pi_tests/pi_stress.c
> @@ -1322,9 +1322,8 @@ int process_sched_line(const char *arg)
> void process_command_line(int argc, char **argv)
> {
> int opt;
> - while ((opt = getopt_long(argc, argv, "+", options, NULL)) != -1) {
> + while ((opt = getopt_long(argc, argv, "+ht:vqi:g:rs:pdVum", options, NULL)) != -1) {
> switch (opt) {
> - case '?':
> case 'h':
> usage();
> exit(0);
> --
> 2.20.1
>
I actually don't see short options in the usage help text, but I do see
them in the manpage.
This could use some clean-up to make sure the manpage is current and the
usage text is current and synced too, if you or anyone else reading cares
to address this.
Signed-off-by: John Kacur <jkacur@xxxxxxxxxx>
