Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-10 Thread Stefan Fritsch
On Thursday 06 January 2011, Plüm, Rüdiger, VF-Group wrote: { -request_rec *rp = apr_pcalloc(r-pool, sizeof(*r)); +apr_pool_t *pool; + +apr_pool_create(pool, c-pool); + +request_rec *rp = apr_pcalloc(pool, sizeof(*r)); MSVC idiosyncrasy,

Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-10 Thread Guenter Knauf
Am 10.01.2011 21:26, schrieb Stefan Fritsch: BTW, gcc has -Wdeclaration-after-statement to catch these things. Should we add that to the default warnings enabled in maintainer mode? +2 - NetWare compiler cant deal with either; and +1 to treat it even as error rather than warning:

Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-05 Thread Graham Leggett
On 05 Jan 2011, at 9:43 AM, Ruediger Pluem wrote: What about the comment in mod_proxy.h about the r element of proxy_conn_rec? /* Request record of the frontend request * which the backend currently answers. */ Doesn't this comment need to be adjusted now? It does - I've fixed it in

RE: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-05 Thread Plüm, Rüdiger, VF-Group
-Original Message- From: Graham Leggett Sent: Mittwoch, 5. Januar 2011 10:30 To: dev@httpd.apache.org Subject: Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c On 05 Jan 2011, at 9:43 AM, Ruediger Pluem wrote

Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-05 Thread Gregg L. Smith
Hello, minf...@apache.org wrote: Author: minfrin Date: Wed Jan 5 00:23:43 2011 New Revision: 1055250 Modified: httpd/httpd/trunk/modules/proxy/proxy_util.c URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/proxy_util.c?rev=1055250r1=1055249r2=1055250view=diff

RE: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-05 Thread Plüm, Rüdiger, VF-Group
-Original Message- From: Gregg L. Smith Sent: Donnerstag, 6. Januar 2011 02:56 To: dev@httpd.apache.org Subject: Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c Hello, wrote: Author: minfrin Date: Wed

Re: svn commit: r1055250 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c

2011-01-04 Thread Ruediger Pluem
On 01/05/2011 01:23 AM, minf...@apache.org wrote: Author: minfrin Date: Wed Jan 5 00:23:43 2011 New Revision: 1055250 URL: http://svn.apache.org/viewvc?rev=1055250view=rev Log: mod_proxy_http: Allocate the fake backend request from a child pool of the backend connection, instead of