On Mon, Jul 30, 2001 at 01:39:56AM +0200, Anton Berezin wrote:
> Hi,
> 
> Graaah!  I have not been reading poe list for a couple of days,
> encountered this same problem, and spent some time fixing it.   :-(((
> Well, tough luck, I guess...
> 
> Anyway.  HTTPD.pm *still* requires some further fixing.  First, it's not
> HTTPD's business to die "Didn't want any more data";  I guess that 400
> Bad Request is in order here as well.

Agreed, and applied.

>                                        Second, I do believe it is not
> HTTPD's business to print anything to stdout - it's great for
> debugging, but does not work well in production.  And third, there are
> some `use of undefined value' occurences there at the moment.

Agreed, agreed, and applied x2.

[... patch ...]

> One minor note:  if ($req->isa('HTTP::Response')) might look better than
> if (ref($req) eq 'HTTP::Response') for OO purists;  I think that the
> docs could mention both forms.

Sounds fair, and I'm checking in a version that does.

-- Rocco Caputo / [EMAIL PROTECTED] / poe.perl.org / poe.sourceforge.net

Reply via email to