Subject: | Move %dates into validate_date() ? |
So this is a really small issue, but since Modern::Perl should be called only once at the beginning of the script, it seems kind of wasteful to keep %dates around by having it package scoped. Maybe it could be moved into validate_date()? It's a small amount of memory, but it just seems unnecessary, and no more complicated to be in the method.