On 9/8/14 8:45 PM, "Karsten Bräckelmann" <guent...@rudersport.de> wrote:
>There is one down side: A new dependency on Regexp::List [1]. The RE >pre-compile one-time upstart penalty should be negligible. > >[1] Well, or a really, really f*cking ugly option that takes a > pre-optimzed qr// blob containing the VALID_TLDS_RE. I may be biased as I've been dealing with a different CPAN dependency flustercluck recently (love maintainers who can't be bothered to update the version info so CPAN doesn't realize there's an update and I have to manually un/re install), but I'm a vote for the hideously ugly preoptimized blob over adding a new dependency. That said, I'd rather have the new dependency than keep the configuration embedded in the rules. So, in order of preference: 1) Pre-optimized blob 2) Regexp::List dependency 3) Current method -- Dave Pooser Cat-Herder-in-Chief, Pooserville.com "...Life is not a journey to the grave with the intention of arriving safely in one pretty and well-preserved piece, but to slide across the finish line broadside, thoroughly used up, worn out, leaking oil, and shouting GERONIMO!!!" -- Bill McKenna
