Re: Tree::Nary Question

in reply to Tree::Nary Question

BrowserUk has shown us what the problem is.

I have a few other observations:

Those messages are warnings, not errors, which means that your code continues to execute despite problems.
It is good that you globally enabled warnings using -w because Tree::Nary does not have use warnings; in its .pm file (a bad thing).
Tree::Nary methods perform no input validation (also a bad thing).

Comment on Re: Tree::Nary Question Select or Download Code

Replies are listed 'Best First'.
Re^2: Tree::Nary Question by Anonymous Monk on Mar 08, 2011 at 20:40 UTC
Yeah, it was a total n00b mistake. Thanks guys!	[reply]
Re^2: Tree::Nary Question by ikegami (Patriarch) on Mar 08, 2011 at 20:52 UTC
It should perform validation to catching things that are not errors? What?	[reply]
Re^3: Tree::Nary Question by Dandello (Monk) on Mar 08, 2011 at 22:04 UTC
Tree::Nary is not checking that the information being passed to it is both valid and not dangerous to run. While I'm not familiar with this module, it's always good to have the information being passed on to be checked for taint and validity.	[reply]
Re^4: Tree::Nary Question by ikegami (Patriarch) on Mar 09, 2011 at 00:19 UTC
It's good cause it catches errors, right? My point was that it's silly to say the messages that result from such errors aren't error messages.	[reply]
Re^5: Tree::Nary Question by Dandello (Monk) on Mar 09, 2011 at 02:02 UTC
Re^6: Tree::Nary Question by ikegami (Patriarch) on Mar 09, 2011 at 07:19 UTC
Some notes below your chosen depth have not been shown here
Re^6: Tree::Nary Question by ikegami (Patriarch) on Mar 09, 2011 at 07:24 UTC

In Section Seekers of Perl Wisdom