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
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 {
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.