On 20 Jul 2015, at 22:17, Junio C Hamano <gits...@pobox.com> wrote: >> +PATTERNS("fountain", >> "^((\\.|(([Ii][Nn][Tt]|[Ee][Ss][Tt]|[Ee][Xx][Tt])?\\.?|[Ii]([Nn][Tt])?\\.?/[Ee]([Xx][Tt])?\\.?) >> ).+)$", >> + "[^ \t-]+"), > > Wouldn't IPATTERN() be a better choice here?
Good point, thank you! Much better: IPATTERN("fountain", "^((\\.|((int|est|ext)?\\.?|i(nt)?\\.?/e(xt)?\\.?) ).+)$", "[^ \t-]+"), It looks like some of the others might benefit from being case insensitive too, but I'm not sure, and at any rate it would warrant a separate patch. I'll send another revision next... :) Thanks, Zoë.-- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html