Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Mo CPAN distribution.

Report information
The Basics
Id: 70915
Status: resolved
Priority: 0/
Queue: Mo

People
Owner: Nobody in particular
Requestors: 'spro^^*%*^6ut# [...] &$%*c
Cc:
AdminCc:

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



Subject: Too verbose
Download (untitled) / with headers
text/plain 325b
This is a nice module, but I find the code rather verbose. Here are some suggestions to make it easier to read: s/no warnings;sub Mo'import{/sub Mo'import{local\$\cW;/ (Yes, I mean use a literal control-W.) s/no \$/no\$/ s/my\$p/\$p/ ($p only ever holds a short string, so leaking it between invocations should be OK.)
Download (untitled) / with headers
text/plain 482b
On Tue Sep 13 02:59:02 2011, SPROUT wrote: Show quoted text
> This is a nice module, but I find the code rather verbose. Here are > some suggestions to make it easier to read: >
Actually, scratch those first two. They won’t work. Show quoted text
> s/no warnings;sub Mo'import{/sub Mo'import{local\$\cW;/ > > (Yes, I mean use a literal control-W.) > > s/no \$/no\$/
This one should still work: Show quoted text
> > s/my\$p/\$p/ > > ($p only ever holds a short string, so leaking it between invocations > should be OK.)
stop by #mo on irc.perl.org and bring your golf clubs. i got a commit bit with your name on it.


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.