diff options
| author | imp <imp@FreeBSD.org> | 1999-04-25 22:37:58 +0000 |
|---|---|---|
| committer | imp <imp@FreeBSD.org> | 1999-04-25 22:37:58 +0000 |
| commit | 22f901a57c1958698d9cecf2b40a9fc0fd04f786 (patch) | |
| tree | 4db883c020b4df70cb765924c2f0a2d0680f4620 /usr.bin/cmp | |
| parent | d12a7ef464b342d4fca0ef83535cf75b401d3f38 (diff) | |
| download | FreeBSD-src-22f901a57c1958698d9cecf2b40a9fc0fd04f786.zip FreeBSD-src-22f901a57c1958698d9cecf2b40a9fc0fd04f786.tar.gz | |
More egcs warning fixes:
o main returns int not void
o use return 0 at end of main when needed
o use braces to avoid potentially ambiguous else
o don't default to type int
o #ifdef 0 -> #if 0
Reviewed by: obrien and chuckr
Diffstat (limited to 'usr.bin/cmp')
| -rw-r--r-- | usr.bin/cmp/regular.c | 3 | ||||
| -rw-r--r-- | usr.bin/cmp/special.c | 6 |
2 files changed, 6 insertions, 3 deletions
diff --git a/usr.bin/cmp/regular.c b/usr.bin/cmp/regular.c index 3b82bae..c5a1e3b 100644 --- a/usr.bin/cmp/regular.c +++ b/usr.bin/cmp/regular.c @@ -93,13 +93,14 @@ c_regular(fd1, file1, skip1, len1, fd2, file2, skip2, len2) p1 += skip1 - off1; p2 += skip2 - off2; for (byte = line = 1; length--; ++p1, ++p2, ++byte) { - if ((ch = *p1) != *p2) + if ((ch = *p1) != *p2) { if (lflag) { dfound = 1; (void)printf("%6qd %3o %3o\n", byte, ch, *p2); } else diffmsg(file1, file2, byte, line); /* NOTREACHED */ + } if (ch == '\n') ++line; } diff --git a/usr.bin/cmp/special.c b/usr.bin/cmp/special.c index b843cc8..a9033b3 100644 --- a/usr.bin/cmp/special.c +++ b/usr.bin/cmp/special.c @@ -73,13 +73,15 @@ c_special(fd1, file1, skip1, fd2, file2, skip2) ch2 = getc(fp2); if (ch1 == EOF || ch2 == EOF) break; - if (ch1 != ch2) + if (ch1 != ch2) { if (lflag) { dfound = 1; (void)printf("%6qd %3o %3o\n", byte, ch1, ch2); - } else + } else { diffmsg(file1, file2, byte, line); /* NOTREACHED */ + } + } if (ch1 == '\n') ++line; } |
