httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject Re: [PATCH] APRize nagle
Date Mon, 21 Aug 2000 14:22:13 GMT

+1.  I would say commit it, and we'll fix any MPMs that we need to make
this work if and when they break.


On Mon, 21 Aug 2000, Brian Havard wrote:

> The current ap_sock_disable_nagle() requires an OS handle. This patch makes 
> it take an apr_socket_t * instead. Note that I had to stick my fingers into 
> MPMs & APR code I can't test which is why I'm posting the patch before 
> commiting it. Is this ok by everyone or do we need to keep the FD_SETSIZE test
> in there? I removed them because this removes the last use of the os socket in
> that part of the code.
> I'm doing this because (a) it's the right thing to do, and (b) is required to
> make OS/2 work right.

Ryan Bloom               
406 29th St.
San Francisco, CA 94131

View raw message