ignore: move star-star matching closer to it use

Instead of threading the state down to the larger loop, let's have the
loop where we detect the double star so each of them are easier to read.
This commit is contained in:
Carlos Martín Nieto 2016-04-18 16:05:12 +02:00
parent 1c3018eb12
commit d45928cc0d

View File

@ -69,8 +69,7 @@ p_fnmatchx(const char *pattern, const char *string, int flags, size_t recurs)
if (recurs-- == 0)
return FNM_NORES;
for (stringstart = string;;) {
bool match_slash = false;
for (stringstart = string;;)
switch (c = *pattern++) {
case EOS:
if ((flags & FNM_LEADING_DIR) && *string == '/')
@ -103,8 +102,15 @@ p_fnmatchx(const char *pattern, const char *string, int flags, size_t recurs)
return (FNM_NOMATCH);
c = *++pattern;
flags &= ~FNM_PATHNAME;
match_slash = true;
do {
int e = p_fnmatchx(pattern, string, recurs_flags, recurs);
if (e != FNM_NOMATCH)
return e;
string = strchr(string, '/');
} while (string++);
/* If we get here, we didn't find a match */
return FNM_NOMATCH;
}
if (*string == '.' && (flags & FNM_PERIOD) &&
@ -135,18 +141,8 @@ p_fnmatchx(const char *pattern, const char *string, int flags, size_t recurs)
return e;
if (test == '/' && (flags & FNM_PATHNAME))
break;
/* searching for star-star, so we jump over entire dirs */
if (match_slash) {
const char *slash;
if (!(slash = strchr(string, '/')))
break;
string = slash + 1;
} else {
++string;
}
}
return (FNM_NOMATCH);
case '[':
if (*string == EOS)
@ -187,7 +183,6 @@ p_fnmatchx(const char *pattern, const char *string, int flags, size_t recurs)
++string;
break;
}
}
/* NOTREACHED */
}