Re: [PATCH] Use PCRE JIT in acl

2013-04-01 Thread Willy Tarreau
Hi, On Wed, Feb 20, 2013 at 03:26:59AM +0900, Hiroaki Nakamura wrote: Hi, Thank you for your review. Sorry for being late, now I made a new patch. As you suggested, I added struct jit_regex. I also add typedef to avoid #ifdef's scatter here and there. I think the code becomes cleaner.

Re: [PATCH] Use PCRE JIT in acl

2013-02-19 Thread Hiroaki Nakamura
Hi, Thank you for your review. Sorry for being late, now I made a new patch. As you suggested, I added struct jit_regex. I also add typedef to avoid #ifdef's scatter here and there. I think the code becomes cleaner. Could you review it? 2013/2/13 Willy Tarreau w...@1wt.eu: Hi, sorry it was

Re: [PATCH] Use PCRE JIT in acl

2013-02-13 Thread Willy Tarreau
Hi, sorry it was long, but I didn't have time to review it earlier. On Sun, Jan 13, 2013 at 03:00:42PM +0900, Hiroaki Nakamura wrote: Hi there. I saw the mail Re: PCRE = 8.20 JIT support in haproxy = 1.5.x? at 2011-12-27, and give it a try. This is a patch for using PCRE JIT in acl.

[PATCH] Use PCRE JIT in acl

2013-01-12 Thread Hiroaki Nakamura
Hi there. I saw the mail Re: PCRE = 8.20 JIT support in haproxy = 1.5.x? at 2011-12-27, and give it a try. This is a patch for using PCRE JIT in acl. Could you review it? I notice regex are used in other places, but they are more complicated to modify to use PCRE APIs. So I focused to acl in