Beefy Boxes and Bandwidth Generously Provided by pair Networks
The stupid question is the question not asked

Re: Re: (Ovid) Re(2): How many bugs can *you* find

by merlyn (Sage)
on May 01, 2001 at 18:30 UTC ( #76969=note: print w/replies, xml ) Need Help??

in reply to Re: (Ovid) Re(2): How many bugs can *you* find
in thread How many bugs can *you* find

Actually, $section can't have regex characters as the code is written, since it simply iterates over the hard-coded items in @sections
As written, but not as maintained. {grin}

For this to pass code review for me, there'd either need to be a fix to that line to make it work even when $section has regex chars, or a note put up above saying "don't ever include regex chars in this string... see below".

The reason I flag that is that it creates an intertwining dependency that will be hell for the maintenance programmer down the road. Trust me: I've been that maintenance programmer enough times to know how much I hate those things. That's why my code reviews are so thorough. {grin}

-- Randal L. Schwartz, Perl hacker

  • Comment on Re: Re: (Ovid) Re(2): How many bugs can *you* find

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://76969]
[james28909]: Good morning!
[james28909]: I have a problem I am trying to solve. I am packaging scripts using pdk and would like the verify the executable name each time it is executed
[Corion]: Hi James!
[james28909]: I looked through the docs, and see that PerlApp::exe() is suppose to return this information but whenever i try to use this, i get the error "Undefined subroutine &PerlApp::exe called"
[james28909]: i just want to verify the integrity of the executables name each time it is run.
[Corion]: Weird. this documentation says what you say.
[Corion]: I assume you are experiencing this with your packaged program. Maybe also include in the packaging list?
[james28909]: has anyone else run into this problem? i have tried searching but have not returned many results
[james28909]: ah, good idea.
[Corion]: What does print $PerlApp::VERSION output?

How do I use this? | Other CB clients
Other Users?
Others cooling their heels in the Monastery: (11)
As of 2017-07-28 13:50 GMT
Find Nodes?
    Voting Booth?
    I came, I saw, I ...

    Results (429 votes). Check out past polls.