Skip Menu |
 

This queue is for tickets about the Net-IMAP-Simple CPAN distribution.

Report information
The Basics
Id: 122709
Status: open
Priority: 0/
Queue: Net-IMAP-Simple

People
Owner: Nobody in particular
Requestors: tlhackque [...] yahoo.com
Cc:
AdminCc:

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



Subject: Net::IMAP::Simple::PipeSocket; can be lazy-loaded
Download (untitled) / with headers
text/plain 434b
N-I-S v 1.2209 There is no point in loading Net::IMAP::Simple::PipeSocket unless it's going to be used. I doubt that using it is a common case. at line 10, delete use Net::IMAP::Simple::PipeSocket; in new() at line 101 if( $server =~ m/cmd:(.+)/ ) { $self->{cmd} = $1; } else { becomes if( $server =~ m/cmd:(.+)/ ) { $self->{cmd} = $1; require Net::IMAP::Simple::PipeSocket; } else {
Download (untitled) / with headers
text/plain 841b
I'm all for it. I know from recent experience I'm not likely to get to it quickly though... Shoot me a PR here if you have the patch all worked out already. https://github.com/jettero/net--imap--simple On Fri Aug 04 14:21:10 2017, tlhackque wrote: Show quoted text
> N-I-S v 1.2209 > > There is no point in loading Net::IMAP::Simple::PipeSocket unless it's > going to be used. I doubt that using it is a common case. > > at line 10, delete > use Net::IMAP::Simple::PipeSocket; > > > in new() at line 101 > if( $server =~ m/cmd:(.+)/ ) { > $self->{cmd} = $1; > > } else { > becomes > if( $server =~ m/cmd:(.+)/ ) { > $self->{cmd} = $1; > require Net::IMAP::Simple::PipeSocket; > } else {
-- If riding in an airplane is flying, then riding in a boat is swimming. 116 jumps, 48.6 minutes of freefall, 92.9 freefall miles.


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.