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 clean

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 : > Hi, > > sorry it was long, but

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 i

[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