Beefy Boxes and Bandwidth Generously Provided by pair Networks
Your skill will accomplish
what the force of many cannot

Re: how did this module get into CPAN ?!

by talexb (Canon)
on Jul 21, 2008 at 15:48 UTC ( #699064=note: print w/replies, xml ) Need Help??

in reply to how did this module get into CPAN ?!

    totally unreadable

Umm .. totally wrong, from the version that I looked at, which was the version packaged with 5.8.7. The indentation is fine, the variable names are reasonable, there are a few comments. An example of or two of the 'unreadable' code would have helped your rant.

    too big compared to what it does

Too big? The version I looked at was 294 lines, broken down into about 75 lines of POD and the remaining code, consisting of six subroutines, and it does a fair bit -- will you be uploading a vastly shorter, 100% backward-compatible version next week?

    contains hidden "logic" that is NOT documented

I didn't see that from my quick scan of the source code -- again, an example or two that supported your claim would have helped here. And (as has already been mentioned) I'm sure the Perl5 Porters would happily accept patches and additional tests if appropriate.

Yes, it's a simple module, but it was written to to do one thing, and one thing well. If it's code that's tried and true, don't sweat it that the code style is a little old. As Edith Bunker once said, "If it's not broken, don't fix it." Words to live by.

Update: Fixed two minor typos.

Alex / talexb / Toronto

"Groklaw is the open-source mentality applied to legal research" ~ Linus Torvalds

  • Comment on Re: how did this module get into CPAN ?!

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://699064]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (9)
As of 2017-12-18 13:54 GMT
Find Nodes?
    Voting Booth?
    What programming language do you hate the most?

    Results (486 votes). Check out past polls.