Beefy Boxes and Bandwidth Generously Provided by pair Networks
good chemistry is complicated,
and a little bit messy -LW
 
PerlMonks  

Re: File::Dependencies - a new module looking for a good name (and more discussion)

by samtregar (Abbot)
on Apr 16, 2002 at 19:10 UTC ( [id://159597]=note: print w/replies, xml ) Need Help??


in reply to File::Dependencies - a new module looking for a good name (and more discussion)

Sounds like a useful module. I thought from the name that it might be some kind of "gcc -M" replacement. I'm glad it's not!

Comments:

  - You commented out the "use 5.006" and then proceeded to   
    use "our".  Naughty.

  - You could get the flexibility you like in Python by
    overloading != in your signature classes, I think.
    I don't know if it would be worth it, but it might
    make you happier with Perl.

  - You have "1;" at the end of every package.  This is not
    required.  What is required is a "1;" at the end of
    every file, which you also have.

  - Why repeat the class name in the "adddependency" method?
    I think just add() might be clearer.

  - The use of mixed case in parameter names runs against
    the grain of most Perl modules ("Method" and "Files").

  - This module might be more useful if it provided 
    save() and load() methods to save dependencies to
    a file and load them again.  That way it could be
    used by compilers and other non-long-running programs
    to track up-to-dateness.  Storable should offer an
    easy solution if you like the idea.

That's all for now!

-sam

  • Comment on Re: File::Dependencies - a new module looking for a good name (and more discussion)

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://159597]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others goofing around in the Monastery: (4)
As of 2024-04-25 12:15 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found