You are viewing a plain text version of this content. The canonical link for it is here.

Posted to cvs@httpd.apache.org by ji...@apache.org on 2006/02/09 15:07:06 UTC

svn commit: r376297 - /httpd/httpd/branches/fcgi-proxy-dev/modules/proxy/mod_proxy_fcgi.c

Author: jim
Date: Thu Feb  9 06:07:03 2006
New Revision: 376297

URL: http://svn.apache.org/viewcvs?rev=376297&view=rev
Log:
Minor style change: IMO checking for APR_SUCCESS is
just better and more consistant.

Modified:
    httpd/httpd/branches/fcgi-proxy-dev/modules/proxy/mod_proxy_fcgi.c

Modified: httpd/httpd/branches/fcgi-proxy-dev/modules/proxy/mod_proxy_fcgi.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/fcgi-proxy-dev/modules/proxy/mod_proxy_fcgi.c?rev=376297&r1=376296&r2=376297&view=diff
==============================================================================
--- httpd/httpd/branches/fcgi-proxy-dev/modules/proxy/mod_proxy_fcgi.c (original)
+++ httpd/httpd/branches/fcgi-proxy-dev/modules/proxy/mod_proxy_fcgi.c Thu Feb  9 06:07:03 2006
@@ -150,7 +150,7 @@
 {
     apr_status_t rv = apr_socket_sendv(conn->sock, vec, nvec, len);
 
-    if (! rv) {
+    if (rv == APR_SUCCESS) {
         conn->worker->s->transferred += *len;
     }
 
@@ -164,7 +164,7 @@
 {
     apr_status_t rv = apr_socket_recv(conn->sock, buffer, buflen);
 
-    if (! rv) {
+    if (rv == APR_SUCCESS) {
         conn->worker->s->read += *buflen;
     }