You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by jf...@apache.org on 2007/08/20 10:33:37 UTC
svn commit: r567596 - /httpd/httpd/branches/2.2.x/STATUS
Author: jfclere
Date: Mon Aug 20 01:33:36 2007
New Revision: 567596
URL: http://svn.apache.org/viewvc?rev=567596&view=rev
Log:
Arrange the proposal to follow rpluem argument.
Modified:
httpd/httpd/branches/2.2.x/STATUS
Modified: httpd/httpd/branches/2.2.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=567596&r1=567595&r2=567596&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Mon Aug 20 01:33:36 2007
@@ -217,14 +217,8 @@
to investigate PR37770.
Trunk version of patch:
http://svn.apache.org/viewvc?view=rev&rev=549420
- +1: jfclere, mturk
- -0.5: rpluem says: As said in
- http://mail-archives.apache.org/mod_mbox/httpd-dev/200706.mbox/%3c46828E5A.50606@apache.org%3e
- I would prefer creating the brigade used in ap_proxygetline outside of
- the loop and pass it to ap_proxygetline for each call. Only do an
- apr_brigade_cleanup inside of ap_proxygetline then. Furthermore the
- check for a buffer overflow a few lines later is not correct any more
- (now an error (APR_ENOSPC) is returned instead).
+ http://svn.apache.org/viewvc?view=rev&rev=561778
+ +1: jfclere
* multiple files, Trivial cleanups
PR: 39518 - Christophe JAILLET