Re: [PATCH] wildmatch test: cover a blind spot in "/" matching

2017-06-10 Thread Ævar Arnfjörð Bjarmason
On Sat, Jun 10 2017, Junio C. Hamano jotted: > Junio C Hamano writes: > >> Ævar Arnfjörð Bjarmason writes: >> >>> Add a test which covers a blindspot in how these tests should assert >>> that negated character classes are allowed or not allowed to match

Re: [PATCH] wildmatch test: cover a blind spot in "/" matching

2017-06-09 Thread Junio C Hamano
Junio C Hamano writes: > Ævar Arnfjörð Bjarmason writes: > >> Add a test which covers a blindspot in how these tests should assert >> that negated character classes are allowed or not allowed to match "/" >> in certain circumstances. > > Please make this a

Re: [PATCH] wildmatch test: cover a blind spot in "/" matching

2017-06-09 Thread Junio C Hamano
Ævar Arnfjörð Bjarmason writes: > Add a test which covers a blindspot in how these tests should assert > that negated character classes are allowed or not allowed to match "/" > in certain circumstances. Please make this a bit more explicit, something like A negated

Re: [PATCH] wildmatch test: cover a blind spot in "/" matching

2017-06-09 Thread Ævar Arnfjörð Bjarmason
On Sat, May 27, 2017 at 11:13 AM, Ævar Arnfjörð Bjarmason wrote: > Add a test which covers a blindspot in how these tests should assert > that negated character classes are allowed or not allowed to match "/" > in certain circumstances. Junio: I didn't CC you on this, because I

[PATCH] wildmatch test: cover a blind spot in "/" matching

2017-05-27 Thread Ævar Arnfjörð Bjarmason
Add a test which covers a blindspot in how these tests should assert that negated character classes are allowed or not allowed to match "/" in certain circumstances. Signed-off-by: Ævar Arnfjörð Bjarmason --- t/t3070-wildmatch.sh | 2 ++ 1 file changed, 2 insertions(+) diff