Skip Menu |
 

This queue is for tickets about the AnyEvent-Handle-UDP CPAN distribution.

Report information
The Basics
Id: 125330
Status: resolved
Priority: 0/
Queue: AnyEvent-Handle-UDP

People
Owner: Nobody in particular
Requestors: alexchandel [...] gmail.com
Cc:
AdminCc:

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



Subject: Remove namespace::clean dependency and usage
Download (untitled) / with headers
text/plain 1.2k
The "namespace::clean" usage has only a single effect, which is to remove the "subname" method: These symbols exist when "namespace::clean" is used and omitted, respectively: BEGIN, VERSION, __ANON__, _bind_to, _connect_to, _drained, _error, _insert, _on_addr, _push_writer, _send, autoflush, bind_to, clear_rtimeout, clear_timeout, clear_wtimeout, connect_to, destroy, fh, new, on_drain, on_error, on_recv, on_rtimeout, on_timeout, on_wtimeout, peername, push_send, receive_size, rtimeout, rtimeout_reset, sockname, timeout, timeout_reset, wtimeout, wtimeout_reset, __NAMESPACE_CLEAN_STORAGE BEGIN, VERSION, __ANON__, _bind_to, _connect_to, _drained, _error, _insert, _on_addr, _push_writer, _send, autoflush, bind_to, clear_rtimeout, clear_timeout, clear_wtimeout, connect_to, destroy, fh, new, on_drain, on_error, on_recv, on_rtimeout, on_timeout, on_wtimeout, peername, push_send, receive_size, rtimeout, rtimeout_reset, sockname, timeout, timeout_reset, wtimeout, wtimeout_reset, subname Why not rename "subname" to "_subname" like the internal "_insert" function, and remove the "namespace::clean" runtime/dependency overhead? You can do `delete ((__PACKAGE__."::")->{subname})` at the end of the module, just before "1;", if you really want to stop the symbol from being exposed.
Subject: Re: [rt.cpan.org #125330] Remove namespace::clean dependency and usage
Date: Wed, 16 May 2018 23:54:04 +0200
To: bug-AnyEvent-Handle-UDP [...] rt.cpan.org
From: Leon Timmermans <leont [...] cpan.org>
Download (untitled) / with headers
text/plain 1.9k
On Wed, May 16, 2018 at 11:30 PM, Alex via RT <bug-AnyEvent-Handle-UDP@rt.cpan.org> wrote: Show quoted text
> Wed May 16 17:30:46 2018: Request 125330 was acted upon. > Transaction: Ticket created by alexchandel@gmail.com > Queue: AnyEvent-Handle-UDP > Subject: Remove namespace::clean dependency and usage > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: alexchandel@gmail.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=125330 > > > > The "namespace::clean" usage has only a single effect, which is to remove the "subname" method: > > These symbols exist when "namespace::clean" is used and omitted, respectively: > > BEGIN, VERSION, __ANON__, _bind_to, _connect_to, _drained, _error, _insert, _on_addr, _push_writer, _send, autoflush, bind_to, clear_rtimeout, clear_timeout, clear_wtimeout, connect_to, destroy, fh, new, on_drain, on_error, on_recv, on_rtimeout, on_timeout, on_wtimeout, peername, push_send, receive_size, rtimeout, rtimeout_reset, sockname, timeout, timeout_reset, wtimeout, wtimeout_reset, __NAMESPACE_CLEAN_STORAGE > > BEGIN, VERSION, __ANON__, _bind_to, _connect_to, _drained, _error, _insert, _on_addr, _push_writer, _send, autoflush, bind_to, clear_rtimeout, clear_timeout, clear_wtimeout, connect_to, destroy, fh, new, on_drain, on_error, on_recv, on_rtimeout, on_timeout, on_wtimeout, peername, push_send, receive_size, rtimeout, rtimeout_reset, sockname, timeout, timeout_reset, wtimeout, wtimeout_reset, subname > > > Why not rename "subname" to "_subname" like the internal "_insert" function, and remove the "namespace::clean" runtime/dependency overhead? > > You can do `delete ((__PACKAGE__."::")->{subname})` at the end of the module, just before "1;", if you really want to stop the symbol from being exposed.
Actually, I had written a patch for this well over a year ago, but apparently I had forgotten to release it. 0.049 is now available on your nearest CPAN mirror ;-) Leon


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.