On 05/12/2009 05:10 PM, jfcl...@apache.org wrote: > Author: jfclere > Date: Tue May 12 15:10:56 2009 > New Revision: 773929 > > URL: http://svn.apache.org/viewvc?rev=773929&view=rev > Log: > Arrange the proxy_balancer_method: > - Put context back after the finder(). > - Use server_rec as second parameter for reset() and age(). > - Add the call for reset(). > > Modified: > httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_bybusyness.c > httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_byrequests.c > httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_bytraffic.c > httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_heartbeat.c > httpd/httpd/trunk/modules/proxy/examples/mod_lbmethod_rr.c > httpd/httpd/trunk/modules/proxy/mod_proxy.h > httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c > > Modified: httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_bybusyness.c > URL: > http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_bybusyness.c?rev=773929&r1=773928&r2=773929&view=diff > ============================================================================== > --- httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_bybusyness.c > (original) > +++ httpd/httpd/trunk/modules/proxy/balancers/mod_lbmethod_bybusyness.c Tue > May 12 15:10:56 2009 > @@ -105,11 +105,11 @@ > > } > > -static apr_status_t reset(proxy_balancer *balancer, request_rec *r) { > +static apr_status_t reset(proxy_balancer *balancer, server_rec *r) {
Could we please rename *r to*s? *r for a server_rec is strange and IMHO wakes wrong associations. Regards RĂ¼diger