in reply to Critique/Test my first module MetaParser
Good work; it's nice to see people contributing new modules (and I could have used a nice prewritten module for this purpose a time or two in the past.) I have a few questions worth considering though.
Why the indirect object notation?
my $parse = new MetaParser;Why create and dereference and rereference a hash, especially when your constructor takes no arguments?
my $obj = {@_}; $obj = bless {%$obj},$pkg || die 'unable to bless object!';
Why would bless fail?
$obj = bless {%$obj},$pkg || die 'unable to bless object!';What benefit is there to making this an object?
|
|---|