From 39c32605072bab8966ff4425f37a158f060bacd2 Mon Sep 17 00:00:00 2001 From: Rob Landley Date: Tue, 26 Jun 2012 20:47:01 -0500 Subject: Minor cleanups and refactoring. Make FLAG macros closer to what should eventually be generated for us by the build infrastructure. --- toys/comm.c | 34 ++++++++++++++-------------------- 1 file changed, 14 insertions(+), 20 deletions(-) diff --git a/toys/comm.c b/toys/comm.c index 1cddf954..1c2267ac 100644 --- a/toys/comm.c +++ b/toys/comm.c @@ -6,11 +6,12 @@ * * See http://pubs.opengroup.org/onlinepubs/009695399/utilities/comm.html -USE_COMM(NEWTOY(comm, "123", TOYFLAG_USR|TOYFLAG_BIN)) +// <# and ># take single digit, so 321 define flags +USE_COMM(NEWTOY(comm, "<2>2321", TOYFLAG_USR|TOYFLAG_BIN)) config COMM bool "comm" - default n + default y help usage: comm [-123] FILE1 FILE2 @@ -25,20 +26,20 @@ config COMM #include "toys.h" -#define FLAG_SUPPRESS_3 1 -#define FLAG_SUPPRESS_2 2 -#define FLAG_SUPPRESS_1 4 +#define FLAG_1 1 +#define FLAG_2 2 +#define FLAG_3 4 static void writeline(const char *line, int col) { - if (col == 0 && toys.optflags & FLAG_SUPPRESS_1) return; + if (col == 0 && toys.optflags & FLAG_1) return; else if (col == 1) { - if (toys.optflags & FLAG_SUPPRESS_2) return; - if (!(toys.optflags & FLAG_SUPPRESS_1)) putchar('\t'); + if (toys.optflags & FLAG_2) return; + if (!(toys.optflags & FLAG_1)) putchar('\t'); } else if (col == 2) { - if (toys.optflags & FLAG_SUPPRESS_3) return; - if (!(toys.optflags & FLAG_SUPPRESS_1)) putchar('\t'); - if (!(toys.optflags & FLAG_SUPPRESS_2)) putchar('\t'); + if (toys.optflags & FLAG_3) return; + if (!(toys.optflags & FLAG_1)) putchar('\t'); + if (!(toys.optflags & FLAG_2)) putchar('\t'); } puts(line); } @@ -49,11 +50,7 @@ void comm_main(void) char *line[2]; int i; - if (toys.optc != 2) - perror_exit("exactly 2 operands required"); - - if (toys.optflags == (FLAG_SUPPRESS_1 | FLAG_SUPPRESS_2 | FLAG_SUPPRESS_3)) - return; + if (toys.optflags == 7) return; for (i = 0; i < 2; i++) { file[i] = strcmp("-", toys.optargs[i]) ? xopen(toys.optargs[i], O_RDONLY) : 0; @@ -84,8 +81,5 @@ void comm_main(void) line[i] = get_line(file[i]); } - if (CFG_TOYBOX_FREE) { - for (i = 0; i < 2; i--) - xclose(file[i]); - } + if (CFG_TOYBOX_FREE) for (i = 0; i < 2; i--) xclose(file[i]); } -- cgit v1.2.3