On Wed, Dec 19, 2012 at 01:30:57PM -0800, Junio C Hamano wrote:

> Jeff King <p...@peff.net> writes:
> 
> > diff --git a/dir.c b/dir.c
> > index 5a83aa7..6e81d4f 100644
> > --- a/dir.c
> > +++ b/dir.c
> > @@ -126,7 +126,7 @@ static int match_one(const char *match, const char 
> > *name, int namelen)
> >             for (;;) {
> >                     unsigned char c1 = tolower(*match);
> >                     unsigned char c2 = tolower(*name);
> > -                   if (c1 == '\0' || is_glob_special(c1))
> > +                   if (c1 == '\0' || (core_pathspec_glob && 
> > is_glob_special(c1)))
> >                             break;
> >                     if (c1 != c2)
> >                             return 0;
> 
> I think you can also do the same to the common_prefix(); we check
> for common leading directory prefix but punt upon seeing a directory
> component that has a glob character, and under the "literal" mode,
> it is not a special character.

Yeah, I think you're right. Will add to my re-roll.

-Peff
--
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

Reply via email to