rpluem at apache
Jul 4, 2009, 12:54 AM
Post #1 of 1
(128 views)
Permalink
|
|
svn commit: r791078 - /httpd/httpd/branches/2.2.x/STATUS
|
|
Author: rpluem
Date: Sat Jul 4 07:54:07 2009
New Revision: 791078
URL: http://svn.apache.org/viewvc?rev=791078&view=rev
Log:
* Add comment and vote.
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=791078&r1=791077&r2=791078&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Sat Jul 4 07:54:07 2009
@@ -92,13 +92,16 @@
Trunk version of patch:
#1 folded in during core output filter refactoring
#2 http://svn.apache.org/viewvc?view=rev&revision=521681
- +1: jorton, jim
+ +1: jorton, jim, rpluem
rpluem asks: Are we sure that b is never NULL?
Otherwise we would need to add
http://svn.apache.org/viewvc?view=rev&revision=568202
as on trunk to avoid segfaults.
trawick responds: if b were NULL, we would have segfaulted earlier
when ap_pass_brigade "calls" APR_BRIGADE_LAST(bb)
+ rpluem: Ahh good point. Meanwhile I had a look at trunk and the
+ event MPM is calling the core output filter directly without
+ ap_pass_brigade. So I am +1.
PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
[ start all new proposals below, under PATCHES PROPOSED. ]
|
