httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r1186545 - /httpd/httpd/trunk/server/mpm/simple/simple_io.c
Date Wed, 26 Oct 2011 12:27:51 GMT

On Oct 25, 2011, at 1:39 PM, William A. Rowe Jr. wrote:

> On 10/25/2011 12:00 PM, Jim Jagielski wrote:
>> These types of things exist for a reason. Please don't
>> remove them…
> 
> Nobody left a comment to explain why it was left dangling,
> and it does nothing.
> 
>   
   /* pqXXXXX: handle timeouts. */

I thought we wanted code that was ez to read and take up
on. The #ifdef shows clearly what that section of code
is supposed to do, but doesn't and serves as an invitation
to someone to fix it.

Remove it and all that is lost.
Mime
View raw message