diff options
| author | phk <phk@FreeBSD.org> | 1997-09-18 14:08:40 +0000 |
|---|---|---|
| committer | phk <phk@FreeBSD.org> | 1997-09-18 14:08:40 +0000 |
| commit | d8ac4091605cafeed429956439b3365036470356 (patch) | |
| tree | 86f85152c10afe3de06afc06d7738c614a3210cb /usr.bin/error | |
| parent | 2d831c7d21d4e39820d9fe862e7ec818476c083b (diff) | |
| download | FreeBSD-src-d8ac4091605cafeed429956439b3365036470356.zip FreeBSD-src-d8ac4091605cafeed429956439b3365036470356.tar.gz | |
Many places in the code NULL is used in integer context, where
plain 0 should be used. This happens to work because we #define
NULL to 0, but is stylistically wrong and can cause problems
for people trying to port bits of code to other environments.
PR: 2752
Submitted by: Arne Henrik Juul <arnej@imf.unit.no>
Diffstat (limited to 'usr.bin/error')
| -rw-r--r-- | usr.bin/error/touch.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.bin/error/touch.c b/usr.bin/error/touch.c index a21c72d..755b313 100644 --- a/usr.bin/error/touch.c +++ b/usr.bin/error/touch.c @@ -595,7 +595,7 @@ writetouched(overwrite) botch = 0; oktorm = 1; - while((nread = fread(edbuf, 1, sizeof(edbuf), o_touchedfile)) != NULL){ + while((nread = fread(edbuf, 1, sizeof(edbuf), o_touchedfile)) != 0) { if (nread != fwrite(edbuf, 1, nread, n_touchedfile)){ /* * Catastrophe in temporary area: file system full? @@ -655,7 +655,7 @@ int mustoverwrite(preciousfile, tmpfile) { int nread; - while((nread = fread(edbuf, 1, sizeof(edbuf), tmpfile)) != NULL){ + while((nread = fread(edbuf, 1, sizeof(edbuf), tmpfile)) != 0) { if (mustwrite(edbuf, nread, preciousfile) == 0) return(0); } |
