Re: [PATCH 8/8] wildmatch: advance faster in + patterns

2012-12-27 Thread Nguyen Thai Ngoc Duy
On Fri, Dec 28, 2012 at 1:24 PM, Junio C Hamano wrote: >> + while ((t_ch = *text) != '\0' && >> +(!(flags & WM_PATHNAME) || t_ch >> != '/')) { > > Why do we look at (flags & WM_PATHMAME) and not "special" here? Becau

Re: [PATCH 8/8] wildmatch: advance faster in + patterns

2012-12-27 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > compat, '*/*/*' on linux-2.6.git file list 2000 times, before: > wildmatch 7s 985049us > fnmatch 2s 735541us or 34.26% faster > > and after: > wildmatch 4s 492549us > fnmatch 0s 888263us or 19.77% slower > > Signed-off-by: Nguyễn Thái Ngọc Duy > --- > wildmat

[PATCH 8/8] wildmatch: advance faster in + patterns

2012-12-21 Thread Nguyễn Thái Ngọc Duy
compat, '*/*/*' on linux-2.6.git file list 2000 times, before: wildmatch 7s 985049us fnmatch 2s 735541us or 34.26% faster and after: wildmatch 4s 492549us fnmatch 0s 888263us or 19.77% slower Signed-off-by: Nguyễn Thái Ngọc Duy --- wildmatch.c | 21 + 1 file changed, 21