Skip to content

Commit c5b22b8

Browse files
committed
Merge branch 'jk/tempfile-ferror-fclose-confusion'
Code clean-up. * jk/tempfile-ferror-fclose-confusion: tempfile: avoid "ferror | fclose" trick
2 parents 62fef5c + 0838cbc commit c5b22b8

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

tempfile.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -247,12 +247,8 @@ int close_tempfile(struct tempfile *tempfile)
247247
tempfile->fd = -1;
248248
if (fp) {
249249
tempfile->fp = NULL;
250-
251-
/*
252-
* Note: no short-circuiting here; we want to fclose()
253-
* in any case!
254-
*/
255-
err = ferror(fp) | fclose(fp);
250+
err = ferror(fp);
251+
err |= fclose(fp);
256252
} else {
257253
err = close(fd);
258254
}

0 commit comments

Comments
 (0)