httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mladen Turk <mt...@apache.org>
Subject Re: svn commit: r682369 - /httpd/httpd/trunk/modules/proxy/proxy_util.c
Date Mon, 04 Aug 2008 13:50:35 GMT
rpluem@apache.org wrote:
> --- httpd/httpd/trunk/modules/proxy/proxy_util.c (original)
> +++ httpd/httpd/trunk/modules/proxy/proxy_util.c Mon Aug  4 05:53:50 2008
> @@ -1380,7 +1380,6 @@
>      proxy_worker *worker = (proxy_worker *)theworker;
>      if (worker->cp->res) {
>          worker->cp->pool = NULL;
> -        apr_reslist_destroy(worker->cp->res);
>      }
>      return APR_SUCCESS;
>  }
>

Well, you've beat me :)
The complete conn_pool_cleanup function can be omitted, and
there is no need for checking  if (conn->worker->cp->pool)
in the connection_destructor.


Regards
-- 
^(TM)

Mime
View raw message