From aa4f108338edeac0c69962aea5607a2c1d5ff0ec Mon Sep 17 00:00:00 2001 From: Mark Whitley Date: Wed, 19 Jul 2000 19:38:13 +0000 Subject: [PATCH] Converted option parsing to getopt() and made some minor formatting changes. --- coreutils/id.c | 39 +++++++++++++++++++++------------------ id.c | 39 +++++++++++++++++++++------------------ 2 files changed, 42 insertions(+), 36 deletions(-) diff --git a/coreutils/id.c b/coreutils/id.c index 56a2e279f..4c524a225 100644 --- a/coreutils/id.c +++ b/coreutils/id.c @@ -33,28 +33,30 @@ extern int id_main(int argc, char **argv) int no_user = 0, no_group = 0, print_real = 0; char *cp, *user, *group; unsigned long gid; + int opt; cp = user = group = NULL; - argc--; argv++; - - while (argc > 0) { - cp = *argv; - if (*cp == '-') { - switch (*++cp) { - case 'u': no_group = 1; break; - case 'g': no_user = 1; break; - case 'r': print_real = 1; break; - default: usage(id_usage); - } - } else { - user = cp; + while ((opt = getopt(argc, argv, "ugr")) > 0) { + switch (opt) { + case 'u': + no_group++; + break; + case 'g': + no_user++; + break; + case 'r': + print_real++; + break; + default: + usage(id_usage); } - argc--; argv++; } if (no_user && no_group) usage(id_usage); + user = argv[optind]; + if (user == NULL) { user = xmalloc(9); group = xmalloc(9); @@ -71,12 +73,13 @@ extern int id_main(int argc, char **argv) my_getgrgid(group, gid); } - if (no_group) printf("%lu\n", my_getpwnam(user)); - else if (no_user) printf("%lu\n", my_getgrnam(group)); + if (no_group) + printf("%lu\n", my_getpwnam(user)); + else if (no_user) + printf("%lu\n", my_getgrnam(group)); else printf("uid=%lu(%s) gid=%lu(%s)\n", - my_getpwnam(user), user, my_getgrnam(group), group); - + my_getpwnam(user), user, my_getgrnam(group), group); return(0); } diff --git a/id.c b/id.c index 56a2e279f..4c524a225 100644 --- a/id.c +++ b/id.c @@ -33,28 +33,30 @@ extern int id_main(int argc, char **argv) int no_user = 0, no_group = 0, print_real = 0; char *cp, *user, *group; unsigned long gid; + int opt; cp = user = group = NULL; - argc--; argv++; - - while (argc > 0) { - cp = *argv; - if (*cp == '-') { - switch (*++cp) { - case 'u': no_group = 1; break; - case 'g': no_user = 1; break; - case 'r': print_real = 1; break; - default: usage(id_usage); - } - } else { - user = cp; + while ((opt = getopt(argc, argv, "ugr")) > 0) { + switch (opt) { + case 'u': + no_group++; + break; + case 'g': + no_user++; + break; + case 'r': + print_real++; + break; + default: + usage(id_usage); } - argc--; argv++; } if (no_user && no_group) usage(id_usage); + user = argv[optind]; + if (user == NULL) { user = xmalloc(9); group = xmalloc(9); @@ -71,12 +73,13 @@ extern int id_main(int argc, char **argv) my_getgrgid(group, gid); } - if (no_group) printf("%lu\n", my_getpwnam(user)); - else if (no_user) printf("%lu\n", my_getgrnam(group)); + if (no_group) + printf("%lu\n", my_getpwnam(user)); + else if (no_user) + printf("%lu\n", my_getgrnam(group)); else printf("uid=%lu(%s) gid=%lu(%s)\n", - my_getpwnam(user), user, my_getgrnam(group), group); - + my_getpwnam(user), user, my_getgrnam(group), group); return(0); }