diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2006-12-22 15:36:13 +0300 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2006-12-22 15:36:13 +0300 |
commit | f93ab47c4560b2a1a3130ac17d0bfb886f09ece8 (patch) | |
tree | 4af247ff3701729e8a9730052ecc675dff911c04 /miscutils | |
parent | 1d14e6f48c266947f78620976697cafc670796f8 (diff) |
time: fix option parsing bug, size optimizations
Diffstat (limited to 'miscutils')
-rw-r--r-- | miscutils/time.c | 50 |
1 files changed, 22 insertions, 28 deletions
diff --git a/miscutils/time.c b/miscutils/time.c index 3f1451c83..7682a2679 100644 --- a/miscutils/time.c +++ b/miscutils/time.c @@ -65,11 +65,12 @@ static const char *const long_format = "\tSocket messages sent: %s\n" "\tSocket messages received: %r\n" "\tSignals delivered: %k\n" - "\tPage size (bytes): %Z\n" "\tExit status: %x"; + "\tPage size (bytes): %Z\n" + "\tExit status: %x"; - /* Wait for and fill in data on child process PID. - Return 0 on error, 1 if ok. */ +/* Wait for and fill in data on child process PID. + Return 0 on error, 1 if ok. */ /* pid_t is short on BSDI, so don't try to promote it. */ static int resuse_end(pid_t pid, resource_t * resp) @@ -110,8 +111,7 @@ static void fprintargv(FILE * fp, char *const *argv, const char *filler) fputs(filler, fp); fputs(*av, fp); } - if (ferror(fp)) - bb_error_msg_and_die(bb_msg_write_error); + die_if_ferror(fp, "output"); } /* Return the number of kilobytes corresponding to a number of pages PAGES. @@ -381,13 +381,11 @@ static void summarize(FILE * fp, const char *fmt, char **command, putc(*fmt++, fp); } - if (ferror(fp)) - bb_error_msg_and_die(bb_msg_write_error); + die_if_ferror(fp, "output"); } putc('\n', fp); - if (ferror(fp)) - bb_error_msg_and_die(bb_msg_write_error); + die_if_ferror(fp, "output"); } /* Run command CMD and return statistics on it. @@ -423,18 +421,16 @@ static void run_command(char *const *cmd, resource_t * resp) int time_main(int argc, char **argv) { - int gotone; resource_t res; const char *output_format = default_format; + char c; - argc--; - argv++; + goto next; /* Parse any options -- don't use getopt() here so we don't * consume the args of our client application... */ - while (argc > 0 && **argv == '-') { - gotone = 0; - while (gotone == 0 && *++(*argv)) { - switch (**argv) { + while (argc > 0 && argv[0][0] == '-') { + while ((c = *++*argv)) { + switch (c) { case 'v': output_format = long_format; break; @@ -444,24 +440,22 @@ int time_main(int argc, char **argv) default: bb_show_usage(); } - argc--; - argv++; - gotone = 1; } + next: + argv++; + argc--; + if (!argc) + bb_show_usage(); } - if (argv == NULL || *argv == NULL) - bb_show_usage(); - run_command(argv, &res); summarize(stderr, output_format, argv, &res); - fflush(stderr); if (WIFSTOPPED(res.waitstatus)) - exit(WSTOPSIG(res.waitstatus)); - else if (WIFSIGNALED(res.waitstatus)) - exit(WTERMSIG(res.waitstatus)); - else if (WIFEXITED(res.waitstatus)) - exit(WEXITSTATUS(res.waitstatus)); + return WSTOPSIG(res.waitstatus); + if (WIFSIGNALED(res.waitstatus)) + return WTERMSIG(res.waitstatus); + if (WIFEXITED(res.waitstatus)) + return WEXITSTATUS(res.waitstatus); return 0; } |