Re: [PATCH] wildmatch test: cover a blind spot in "/" matching
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 10, 2017, 01:57 UTC
- Message-ID
- <xmqqh8zo4na3.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <xmqqlgp04ne3.fsf@gitster.mtv.corp.google.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 10 quoted lines
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> 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 character class that does not include '/', > e.g. [^a-z],
oops. I think the class includes '/'; please excuse loose logic in a "something like" meant as an illustration to show the level of details expected ;-)