You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by tr...@apache.org on 2013/11/16 13:13:40 UTC
svn commit: r1542493 - /httpd/httpd/trunk/modules/lua/lua_request.c
Author: trawick
Date: Sat Nov 16 12:13:39 2013
New Revision: 1542493
URL: http://svn.apache.org/r1542493
Log:
apr_file_write_full() simplification (like r1542413 and r1542416)
hopefully I sidestepped the surprising conflation of OK/APR_SUCCESS
in this code
Modified:
httpd/httpd/trunk/modules/lua/lua_request.c
Modified: httpd/httpd/trunk/modules/lua/lua_request.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/lua/lua_request.c?rev=1542493&r1=1542492&r2=1542493&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/lua/lua_request.c (original)
+++ httpd/httpd/trunk/modules/lua/lua_request.c Sat Nov 16 12:13:39 2013
@@ -229,7 +229,6 @@ static apr_status_t lua_write_body(reque
len_read,
rpos = 0;
apr_off_t length = r->remaining;
- apr_size_t written;
*size = length;
while ((len_read =
@@ -241,9 +240,9 @@ static apr_status_t lua_write_body(reque
rsize = len_read;
rc = apr_file_write_full(file, argsbuffer, (apr_size_t) rsize,
- &written);
- if (written != rsize || rc != OK)
- return APR_ENOSPC;
+ NULL);
+ if (rc != APR_SUCCESS)
+ return rc;
rpos += rsize;
}
}