You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Ruediger Pluem <rp...@apache.org> on 2009/05/12 20:53:16 UTC
Re: svn commit: r773929 - in /httpd/httpd/trunk/modules/proxy: ./
balancers/ examples/
On 05/12/2009 05:10 PM, jfclere@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
Re: svn commit: r773929 - in /httpd/httpd/trunk/modules/proxy: ./ balancers/ examples/
Posted by Jim Jagielski <ji...@jaguNET.com>.
On May 12, 2009, at 2:53 PM, Ruediger Pluem wrote:
>
>
> On 05/12/2009 05:10 PM, jfclere@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.
>
That's why I didn't notice it :)