On Tue, 7 Aug 2007, Junio C Hamano wrote:
>
> > @@ -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 ;-).
Gaah, that was unintentional. Just random noise I had in my tree, and
didn't even realize made it into the patch.
That "close_or_die()" was because I saw somebody report a write error
without the error string, apparently because the error only got reported
on the close (probably NFS). This way you see if the reason the close
failed was due to out of diskspace or whatever.
But I should have split them up properly - the close_or_die() part
obviously had nothing to do with the O_BINARY part. Feel free to take it
regardless, or split it yourself.
Linus
-
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
