diff options
| author | Jeff King <peff@peff.net> | 2018-01-11 01:31:10 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2019-01-03 14:15:32 -0800 |
| commit | d4c813689b2b320f8907a615fcf1f1d27ae44f37 (patch) | |
| tree | 4d8aa5d5caa91b8be1a9f72d5cfb9f08aa024a42 /transport-helper.c | |
| parent | c14e5a1a5018f4407390488dfea387e5dc81c817 (diff) | |
| download | git-d4c813689b2b320f8907a615fcf1f1d27ae44f37.tar.gz | |
transport-helper: drop read/write errno checks
Since we use xread() and xwrite() here, EINTR, EAGAIN, and
EWOULDBLOCK retries are already handled for us, and we will
never see these errno values ourselves. We can drop these
conditions entirely, making the code easier to follow.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'transport-helper.c')
| -rw-r--r-- | transport-helper.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/transport-helper.c b/transport-helper.c index a290695a12..a87fdf7b0b 100644 --- a/transport-helper.c +++ b/transport-helper.c @@ -1226,8 +1226,7 @@ static int udt_do_read(struct unidirectional_transfer *t) transfer_debug("%s is readable", t->src_name); bytes = xread(t->src, t->buf + t->bufuse, BUFFERSIZE - t->bufuse); - if (bytes < 0 && errno != EWOULDBLOCK && errno != EAGAIN && - errno != EINTR) { + if (bytes < 0) { error_errno(_("read(%s) failed"), t->src_name); return -1; } else if (bytes == 0) { @@ -1254,7 +1253,7 @@ static int udt_do_write(struct unidirectional_transfer *t) transfer_debug("%s is writable", t->dest_name); bytes = xwrite(t->dest, t->buf, t->bufuse); - if (bytes < 0 && errno != EWOULDBLOCK) { + if (bytes < 0) { error_errno(_("write(%s) failed"), t->dest_name); return -1; } else if (bytes > 0) { |