On Mon, 2014-09-08 at 21:45 -0500, Dave Pooser wrote: > 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. ^^^^^ Code, not rules. Which basically is the issue here...
> So, in order of preference: > 1) Pre-optimized blob > 2) Regexp::List dependency > 3) Current method Got ya. Both (1) and (2) would require code changes, so it's 3.4.1+ only anyway. Thanks. -- char *t="\10pse\0r\0dtu\0.@ghno\x4e\xc8\x79\xf4\xab\x51\x8a\x10\xf4\xf4\xc4"; main(){ char h,m=h=*t++,*x=t+2*h,c,i,l=*x,s=0; for (i=0;i<l;i++){ i%8? c<<=1: (c=*++x); c&128 && (s+=h); if (!(h>>=1)||!t[s+h]){ putchar(t[s]);h=m;s=0; }}}