diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2018-11-21 18:37:44 -0800 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2018-11-21 18:38:20 -0800 |
commit | ad063d2552d4d31fa668fa5f15a91aec18c010f6 (patch) | |
tree | 6d1ca54e87f89e28c55894e98af26a3ec493b391 | |
parent | 0f22bf099e569790fe6bb830522b5e41be41bbb6 (diff) | |
download | emacs-ad063d2552d4d31fa668fa5f15a91aec18c010f6.tar.gz |
emacsclient: getopt minor cleanup
* lib-src/emacsclient.c (shortopts): New constant.
(decode_options): Use it. Do not assume EOF == -1.
-rw-r--r-- | lib-src/emacsclient.c | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/lib-src/emacsclient.c b/lib-src/emacsclient.c index 908602ec254..4ab97c3c50c 100644 --- a/lib-src/emacsclient.c +++ b/lib-src/emacsclient.c @@ -153,6 +153,7 @@ static char const *frame_parameters; static _Noreturn void print_help_and_exit (void); +/* Long command-line options. */ static struct option const longopts[] = { @@ -177,6 +178,15 @@ static struct option const longopts[] = { 0, 0, 0, 0 } }; +/* Short options, in the same order as the corresponding long options. + There is no '-p' short option. */ +static char const shortopts[] = + "nqueHVtca:F:" +#ifndef NO_SOCKETS_IN_FILE_SYSTEM + "s:" +#endif + "f:d:T:"; + /* Like malloc but get fatal error if memory is exhausted. */ @@ -485,15 +495,8 @@ decode_options (int argc, char **argv) while (true) { - int opt = getopt_long_only (argc, argv, -#ifndef NO_SOCKETS_IN_FILE_SYSTEM - "VHnequa:s:f:d:F:tcT:", -#else - "VHnequa:f:d:F:tcT:", -#endif - longopts, 0); - - if (opt == EOF) + int opt = getopt_long_only (argc, argv, shortopts, longopts, NULL); + if (opt < 0) break; switch (opt) |