On 11/06/2017 02:23 AM, Junio C Hamano wrote: > Michael Haggerty <mhagger@xxxxxxxxxxxx> writes: > >> [1] I say "almost entirely" because putting them in one function means >> that only `pattern` needs to be scanned for glob characters. But that is >> an unimportant detail. > > That could actually be an important detail, in that even if prefix > has wildcard, we'd still append the trailing "/*" as long as the > pattern does not, right? That's correct, but I was assuming that the prefix would always be a hard-coded string like "refs/tags/" or maybe "refs/". (That is the case now.) It doesn't seem very useful to use a prefix like "refs/*/". Michael
