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)

History Show all quoted text
  Thu Mar 20 13:14:56 2008 diz [...] cpan.org - Ticket created
Subject: no way to turn off debugging in POE::Wheel::SocketFactory
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.
  Sat Apr 19 02:22:02 2008 RCAPUTO [...] cpan.org - Correspondence added
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).
  Sat Apr 19 02:22:03 2008 The RT System itself - Status changed from 'new' to 'open'
  Sat Apr 19 02:22:04 2008 RCAPUTO [...] cpan.org - Status changed from 'open' to 'rejected'