Skip to content

Commit

Permalink
Merge branch 'ef/mingw-write'
Browse files Browse the repository at this point in the history
* ef/mingw-write:
  mingw: remove mingw_write
  prefer xwrite instead of write
  • Loading branch information
Junio C Hamano committed Jan 27, 2014
2 parents de20e44 + c9df6f4 commit 7b4e2b7
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 25 deletions.
2 changes: 1 addition & 1 deletion builtin/merge.c
Original file line number Diff line number Diff line change
Expand Up @@ -367,7 +367,7 @@ static void squash_message(struct commit *commit, struct commit_list *remotehead
sha1_to_hex(commit->object.sha1));
pretty_print_commit(&ctx, commit, &out);
}
if (write(fd, out.buf, out.len) < 0)
if (write_in_full(fd, out.buf, out.len) != out.len)
die_errno(_("Writing SQUASH_MSG"));
if (close(fd))
die_errno(_("Finishing SQUASH_MSG"));
Expand Down
17 changes: 0 additions & 17 deletions compat/mingw.c
Original file line number Diff line number Diff line change
Expand Up @@ -304,23 +304,6 @@ int mingw_open (const char *filename, int oflags, ...)
return fd;
}

#undef write
ssize_t mingw_write(int fd, const void *buf, size_t count)
{
/*
* While write() calls to a file on a local disk are translated
* into WriteFile() calls with a maximum size of 64KB on Windows
* XP and 256KB on Vista, no such cap is placed on writes to
* files over the network on Windows XP. Unfortunately, there
* seems to be a limit of 32MB-28KB on X64 and 64MB-32KB on x86;
* bigger writes fail on Windows XP.
* So we cap to a nice 31MB here to avoid write failures over
* the net without changing the number of WriteFile() calls in
* the local case.
*/
return write(fd, buf, min(count, 31 * 1024 * 1024));
}

static BOOL WINAPI ctrl_ignore(DWORD type)
{
return TRUE;
Expand Down
3 changes: 0 additions & 3 deletions compat/mingw.h
Original file line number Diff line number Diff line change
Expand Up @@ -180,9 +180,6 @@ int mingw_rmdir(const char *path);
int mingw_open (const char *filename, int oflags, ...);
#define open mingw_open

ssize_t mingw_write(int fd, const void *buf, size_t count);
#define write mingw_write

int mingw_fgetc(FILE *stream);
#define fgetc mingw_fgetc

Expand Down
2 changes: 1 addition & 1 deletion streaming.c
Original file line number Diff line number Diff line change
Expand Up @@ -538,7 +538,7 @@ int stream_blob_to_fd(int fd, unsigned const char *sha1, struct stream_filter *f
goto close_and_exit;
}
if (kept && (lseek(fd, kept - 1, SEEK_CUR) == (off_t) -1 ||
write(fd, "", 1) != 1))
xwrite(fd, "", 1) != 1))
goto close_and_exit;
result = 0;

Expand Down
5 changes: 2 additions & 3 deletions transport-helper.c
Original file line number Diff line number Diff line change
Expand Up @@ -1135,9 +1135,8 @@ static int udt_do_write(struct unidirectional_transfer *t)
return 0; /* Nothing to write. */

transfer_debug("%s is writable", t->dest_name);
bytes = write(t->dest, t->buf, t->bufuse);
if (bytes < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&
errno != EINTR) {
bytes = xwrite(t->dest, t->buf, t->bufuse);
if (bytes < 0 && errno != EWOULDBLOCK) {
error("write(%s) failed: %s", t->dest_name, strerror(errno));
return -1;
} else if (bytes > 0) {
Expand Down

0 comments on commit 7b4e2b7

Please sign in to comment.