Skip Menu |
 

This queue is for tickets about the POE CPAN distribution.

Report information
The Basics
Id: 34288
Status: rejected
Priority: 0/
Queue: POE

People
Owner: Nobody in particular
Requestors: diz [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.9999
Fixed in: (no value)



Subject: no way to turn off debugging in POE::Wheel::SocketFactory
Download (untitled) / with headers
text/plain 376b
the DEBUG sub in POE::Wheel::SocketFactory returns 1 in the default distribution and has a prototype of (), meaning it gets inlined. this prevents it from being overridden in any way. i'd recommend that the () prototype be removed so that it can be overridden or, better yet, do what POE::Component::Server::SOAP does and only define it if it hasn't been previously defined.
Download (untitled) / with headers
text/plain 645b
DEBUG is off by default. You really want a convenient way to turn it on. And possibly a way to toggle it at runtime. Runtime changes would require extra CPU to test the flag. At least 99% of use cases would be needlessly penalized for the convenience, so I'm against this. I'm also not inclined to make the constant friendlier to change at compile time. In the few cases where debugging SocketFactory is significant, the developer can just edit the source. Also, TRACE_EVENTS (see perldoc POE::Kernel) will also trace SocketFactory events, so it's good enough for most uses. As does the "trace" session option (see perldoc POE::Session).


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.