Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> writes: > On Tue, 7 Aug 2007, Steffen Prohaska wrote: >> >> Is there any chance that patches would be accepted that try to >> do so? Even if they add "b" to fopen and O_BINARY to open, which >> both are useless on Unix? > > I certainly don't think it would be wrong to add O_BINARY to the open() > parameters (and "b" to fopen() and friends), if it makes a difference. > > Add a > > #ifndef O_BINARY > #define O_BINARY 0 > #endif > > and it should be harmless anywhere else. Heh, you beat me to this --- I like it. > @@ -2023,6 +2027,12 @@ int move_temp_to_file(const char *tmpfile, const char *filename) > ... > +static void close_or_die(int fd, const char *file) > +{ > + if (close(fd)) > + die("unable to close %s (%s)", file, strerror(errno)); > +} > + And I like this even better ;-). - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html