diff options
author | Mike Frysinger <vapier@gentoo.org> | 2005-05-13 02:50:12 +0400 |
---|---|---|
committer | Mike Frysinger <vapier@gentoo.org> | 2005-05-13 02:50:12 +0400 |
commit | dad4cf7e6307b5b6032d928f0c897235b3e1042d (patch) | |
tree | 7e7366348fe8c8dab7eebad25cfdd1adad8ba460 /coreutils | |
parent | b3a6ec3e623d120eee39bd003c8efdff85c0497a (diff) |
use a bunch of if statements since it is a few bytes smaller than a switch; also use bb_xfopen() instead of fopen() so comm doesnt segfault when given non-existant files :(
Diffstat (limited to 'coreutils')
-rw-r--r-- | coreutils/comm.c | 36 |
1 files changed, 16 insertions, 20 deletions
diff --git a/coreutils/comm.c b/coreutils/comm.c index 2354bac0b..b0384fe87 100644 --- a/coreutils/comm.c +++ b/coreutils/comm.c @@ -40,25 +40,21 @@ static int both; /* writeline outputs the input given, appropriately aligned according to class */ static void writeline(char *line, int class) { - switch (class) { - case 1: - if (!only_file_1) - return; - break; - case 2: - if (!only_file_2) - return; - if (only_file_1) - putchar('\t'); - break; - case 3: - if (!both) - return; - if (only_file_1) - putchar('\t'); - if (only_file_2) - putchar('\t'); - break; + if (class == 1 && !only_file_1) + return; + else if (class == 2) { + if (!only_file_2) + return; + if (only_file_1) + putchar('\t'); + } + else if (class == 3) { + if (!both) + return; + if (only_file_1) + putchar('\t'); + if (only_file_2) + putchar('\t'); } fputs(line, stdout); } @@ -71,7 +67,7 @@ static int cmp_files(char **infiles) int i = 0; for (i = 0; i < 2; i++) { - streams[i] = (strcmp(infiles[i], "=") == 0 ? stdin : fopen(infiles[i], "r")); + streams[i] = (strcmp(infiles[i], "=") == 0 ? stdin : bb_xfopen(infiles[i], "r")); fgets(thisline[i], 100, streams[i]); } |