Hi Peff & Junio, On Thu, 8 Sep 2016, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > > Between the two options for regexec_buf(), I think you have convinced me > > that REG_STARTEND is better than just using compat/regex everywhere. I > > do think the fallback for platforms like musl should be "use > > compat/regex" and not doing an expensive copy (which in most cases is > > not even necessary). > > I agree with you that it would be the best approach to build > regexec_buf() that unconditionally uses REG_STARTEND and tell people > without REG_STARTEND to use compat/regex instead of their platform > regex library. > > The description in Makefile may want to be rephrased to clarify. > > -# Define NO_REGEX if you have no or inferior regex support in your C library. > +# Define NO_REGEX if your C library lacks regex support with REG_STARTEND > +# feature. Okay, I will make that happen. But first I need to sleep. Ciao, Dscho