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