diff options
author | Denys Vlasenko | 2017-08-09 19:20:08 +0200 |
---|---|---|
committer | Denys Vlasenko | 2017-08-09 19:20:08 +0200 |
commit | 7943be1e13d6c09f2adda5e954d56ff019a79008 (patch) | |
tree | f76ad8861a5b2ab0b38da46de121df4dc8111689 | |
parent | a4d4ab04c3706af1cb2f65fb00c96d3ccba1020c (diff) | |
download | busybox-7943be1e13d6c09f2adda5e954d56ff019a79008.zip busybox-7943be1e13d6c09f2adda5e954d56ff019a79008.tar.gz |
ipcrm: code shrink
function old new delta
ipcrm_main 698 663 -35
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | util-linux/ipcrm.c | 40 |
1 files changed, 16 insertions, 24 deletions
diff --git a/util-linux/ipcrm.c b/util-linux/ipcrm.c index c51d331..fa1159f 100644 --- a/util-linux/ipcrm.c +++ b/util-linux/ipcrm.c @@ -19,15 +19,6 @@ //kbuild:lib-$(CONFIG_IPCRM) += ipcrm.o -//usage:#define ipcrm_trivial_usage -//usage: "[-MQS key] [-mqs id]" -//usage:#define ipcrm_full_usage "\n\n" -//usage: "Upper-case options MQS remove an object by shmkey value.\n" -//usage: "Lower-case options remove an object by shmid value.\n" -//usage: "\n -mM Remove memory segment after last detach" -//usage: "\n -qQ Remove message queue" -//usage: "\n -sS Remove semaphore" - #include "libbb.h" /* X/OPEN tells us to use <sys/{types,ipc,sem}.h> for semctl() */ @@ -94,6 +85,14 @@ static int remove_ids(type_id type, char **argv) } #endif /* IPCRM_LEGACY */ +//usage:#define ipcrm_trivial_usage +//usage: "[-MQS key] [-mqs id]" +//usage:#define ipcrm_full_usage "\n\n" +//usage: "Upper-case options MQS remove an object by shmkey value.\n" +//usage: "Lower-case options remove an object by shmid value.\n" +//usage: "\n -mM Remove memory segment after last detach" +//usage: "\n -qQ Remove message queue" +//usage: "\n -sS Remove semaphore" int ipcrm_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int ipcrm_main(int argc, char **argv) @@ -137,28 +136,20 @@ int ipcrm_main(int argc, char **argv) #endif /* IPCRM_LEGACY */ /* process new syntax to conform with SYSV ipcrm */ - while ((c = getopt(argc, argv, "q:m:s:Q:M:S:h?")) != -1) { + while ((c = getopt(argc, argv, "q:m:s:Q:M:S:")) != -1) { int result; - int id = 0; - int iskey = isupper(c); - + int id; + int iskey; /* needed to delete semaphores */ union semun arg; - arg.val = 0; - - if ((c == '?') || (c == 'h')) { + if (c == '?') /* option not in the string */ bb_show_usage(); - } - - /* we don't need case information any more */ - c = tolower(c); - /* make sure the option is in range: allowed are q, m, s */ - if (c != 'q' && c != 'm' && c != 's') { - bb_show_usage(); - } + id = 0; + arg.val = 0; + iskey = !(c & 0x20); /* uppercase? */ if (iskey) { /* keys are in hex or decimal */ key_t key = xstrtoul(optarg, 0); @@ -169,6 +160,7 @@ int ipcrm_main(int argc, char **argv) continue; } + c |= 0x20; /* lowercase. c is 'q', 'm' or 's' now */ /* convert key to id */ id = ((c == 'q') ? msgget(key, 0) : (c == 'm') ? shmget(key, 0, 0) : semget(key, 0, 0)); |