Use NULL instead of 0 for the return value of fopen().

Approved by: re (kensmith)
This commit is contained in:
Kevin Lo 2007-09-21 01:55:11 +00:00
parent 65db91329e
commit 8bcd62f2ff
4 changed files with 6 additions and 6 deletions

View File

@ -184,7 +184,7 @@ nextjob()
long jobno;
FILE *fid;
if ((fid = fopen(ATJOB_DIR ".SEQ", "r+")) != (FILE*)0) {
if ((fid = fopen(ATJOB_DIR ".SEQ", "r+")) != NULL) {
if (fscanf(fid, "%5lx", &jobno) == 1) {
rewind(fid);
jobno = (1+jobno) % 0xfffff; /* 2^20 jobs enough? */
@ -195,7 +195,7 @@ nextjob()
fclose(fid);
return jobno;
}
else if ((fid = fopen(ATJOB_DIR ".SEQ", "w")) != (FILE*)0) {
else if ((fid = fopen(ATJOB_DIR ".SEQ", "w")) != NULL) {
fprintf(fid, "%05lx\n", jobno = 1);
fclose(fid);
return 1;

View File

@ -93,8 +93,8 @@ gid_t gid;
int prot;
const char *table;
if ((fp = fopen(table = _PATH_FBTAB, "r")) == 0
&& (fp = fopen(table = _PATH_LOGINDEVPERM, "r")) == 0)
if ((fp = fopen(table = _PATH_FBTAB, "r")) == NULL
&& (fp = fopen(table = _PATH_LOGINDEVPERM, "r")) == NULL)
return;
while (fgets(buf, sizeof(buf), fp)) {

View File

@ -133,7 +133,7 @@ dopath(struct input_file *i, const char *filename)
for (pe = first; pe; pe = pe->next) {
snprintf(path, sizeof(path), "%s/%s", pe->name, filename);
if ((f = fopen(path, "r")) != 0) {
if ((f = fopen(path, "r")) != NULL) {
set_input(i, f, path);
return i;
}

View File

@ -234,7 +234,7 @@ main(int ac, char *av[])
break;
case 'o':
locale_file = optarg;
if ((fp = fopen(locale_file, "w")) == 0)
if ((fp = fopen(locale_file, "w")) == NULL)
err(1, "%s", locale_file);
atexit(cleanout);
break;