summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjerenkrantz <jerenkrantz@13f79535-47bb-0310-9956-ffa450edef68>2001-10-10 15:24:03 +0000
committerjerenkrantz <jerenkrantz@13f79535-47bb-0310-9956-ffa450edef68>2001-10-10 15:24:03 +0000
commitbe30689b1b25a2b91291a01aa8adafa9463b8e42 (patch)
treee485cac0133591d63d3f6b04bcfec236d7078ced
parent07b9506a284d4075074430bc9533210eaaf4573d (diff)
downloadlibapr-util-be30689b1b25a2b91291a01aa8adafa9463b8e42.tar.gz
Fair enough. =-)
Ryan and Cliff have convinced me that EAGAIN shouldn't be handled here. It probably needs to be documented that buckets can return EAGAIN or APR_SUCCESS. git-svn-id: http://svn.apache.org/repos/asf/apr/apr-util/trunk@58430 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r--buckets/apr_buckets_socket.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/buckets/apr_buckets_socket.c b/buckets/apr_buckets_socket.c
index ab132a88..59148d6b 100644
--- a/buckets/apr_buckets_socket.c
+++ b/buckets/apr_buckets_socket.c
@@ -76,12 +76,6 @@ static apr_status_t socket_read(apr_bucket *a, const char **str,
if (block == APR_NONBLOCK_READ) {
apr_setsocketopt(p, APR_SO_TIMEOUT, timeout);
- /* There was nothing to read right now, so treat it as okay and
- * return a 0-length brigade (see below). */
- if (APR_STATUS_IS_EAGAIN(rv)) {
- *len = 0;
- rv = APR_SUCCESS;
- }
}
if (rv != APR_SUCCESS && rv != APR_EOF) {