Re: [PATCH 2/2] wildmatch: use the new precompiling wildmatch()

2018-02-26 Thread Ævar Arnfjörð Bjarmason
On Mon, Feb 26 2018, Duy Nguyen jotted: > On Mon, Feb 26, 2018 at 3:35 AM, Ævar Arnfjörð Bjarmason > wrote: >> diff --git a/config.c b/config.c >> index b0c20e6cb8..0f595de971 100644 >> --- a/config.c >> +++ b/config.c >> @@ -210,6 +210,7 @@ static int include_by_gitdir(const

Re: [PATCH 2/2] wildmatch: use the new precompiling wildmatch()

2018-02-26 Thread Duy Nguyen
On Mon, Feb 26, 2018 at 3:35 AM, Ævar Arnfjörð Bjarmason wrote: > diff --git a/config.c b/config.c > index b0c20e6cb8..0f595de971 100644 > --- a/config.c > +++ b/config.c > @@ -210,6 +210,7 @@ static int include_by_gitdir(const struct config_options > *opts, > int ret =

[PATCH 2/2] wildmatch: use the new precompiling wildmatch()

2018-02-25 Thread Ævar Arnfjörð Bjarmason
Make some limited use of the wildmatch() interface for "precompiling" patterns, although the current implementation does not do this yet. The main hot codepath in dir.c is not being touched yet, but some other invocations where we repeatedly match the same glob against multiple strings have been