Beefy Boxes and Bandwidth Generously Provided by pair Networks
We don't bite newbies here... much

Re: MessageLibrary

by mkmcconn (Chaplain)
on Jan 07, 2002 at 11:42 UTC ( #136793=note: print w/replies, xml ) Need Help??

in reply to MessageLibrary

I like your MessageLibrary, seattlejohn. It works as advertised, and I think it would be quite useful for a larger project.

In the humble hope of offering something more useful to you than an 'attaboy': You put a lot of work into the pod, and evidently want it to be as informative as possible; still, I found the inline pod very difficult to read around while trying to see what the code was doing (until I loaded the source in a highlighting editor).

Also, it might be helpful if your comments on how to use it were even more clearly distinguished from what's happening inside the program (for example, I'm thinking here of the pod comments on Private Methods and variables).

Thank you for sharing your work, here.

Replies are listed 'Best First'.
Re: Re: MessageLibrary
by seattlejohn (Deacon) on Jan 07, 2002 at 12:35 UTC
    Thanks for the feedback, mkmcconn. Your comment about the inline pod doesn't come as a complete surprise; it occurred to me before posting that this could be seen as an impediment to following the code flow. I must admit that I routinely use a highlighting editor and kind of like having the documentation right next to the code it applies to, partly because I find it's easier to keep things in sync that way. But given your comment (and my suspicion that for everyone who comments there are probably ten who feel the same way but can't be bothered to respond), I'll definitely reconsider. cheers

Log In?

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

How do I use this? | Other CB clients
Other Users?
Others exploiting the Monastery: (5)
As of 2016-10-01 05:14 GMT
Find Nodes?
    Voting Booth?
    Extraterrestrials haven't visited the Earth yet because:

    Results (575 votes). Check out past polls.