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; }}}

Reply via email to