Good stuff.
Although I vote that you change the default behavior to not croak
(assuming you go with the attribute on the constructor). I
realize this is not exactly backwards compatible but my reasoning is
as follows:
- The standard perl idiom is:
my $dt = $strp->parse_datetime()
or die;
not:
my $dt = eval { $strp->parse_datetime(); };
die $@ if $@;
- If you have warnings and strict on then if parse_datetime returns
undef you will be told about it
Anyway, if you do make the error handling method part of the object
rather than a package global, you should do the same thing with the
error string by making a method to get the error test for the last
error on the object.
Thanks,
-ben
