Skip to content

Commit

Permalink
wildmatch: remove static variable force_lower_case
Browse files Browse the repository at this point in the history
One place less to worry about thread safety. Also combine wildmatch
and iwildmatch into one.

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Nguyễn Thái Ngọc Duy authored and Junio C Hamano committed Oct 15, 2012
1 parent 3ae5396 commit 9b4edc0
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 20 deletions.
4 changes: 2 additions & 2 deletions test-wildmatch.c
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@
int main(int argc, char **argv)
{
if (!strcmp(argv[1], "wildmatch"))
return !!wildmatch(argv[3], argv[2]);
return !!wildmatch(argv[3], argv[2], 0);
else if (!strcmp(argv[1], "iwildmatch"))
return !!iwildmatch(argv[3], argv[2]);
return !!wildmatch(argv[3], argv[2], FNM_CASEFOLD);
else if (!strcmp(argv[1], "fnmatch"))
return !!fnmatch(argv[3], argv[2], FNM_PATHNAME);
else
Expand Down
21 changes: 5 additions & 16 deletions wildmatch.c
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,8 @@ typedef unsigned char uchar;
#define ISUPPER(c) (ISASCII(c) && isupper(c))
#define ISXDIGIT(c) (ISASCII(c) && isxdigit(c))

static int force_lower_case = 0;

/* Match pattern "p" against "text" */
static int dowild(const uchar *p, const uchar *text)
static int dowild(const uchar *p, const uchar *text, int force_lower_case)
{
uchar p_ch;

Expand Down Expand Up @@ -106,7 +104,7 @@ static int dowild(const uchar *p, const uchar *text)
while (1) {
if (t_ch == '\0')
break;
if ((matched = dowild(p, text)) != NOMATCH) {
if ((matched = dowild(p, text, force_lower_case)) != NOMATCH) {
if (!special || matched != ABORT_TO_STARSTAR)
return matched;
} else if (!special && t_ch == '/')
Expand Down Expand Up @@ -214,17 +212,8 @@ static int dowild(const uchar *p, const uchar *text)
}

/* Match the "pattern" against the "text" string. */
int wildmatch(const char *pattern, const char *text)
{
return dowild((const uchar*)pattern, (const uchar*)text);
}

/* Match the "pattern" against the forced-to-lower-case "text" string. */
int iwildmatch(const char *pattern, const char *text)
int wildmatch(const char *pattern, const char *text, int flags)
{
int ret;
force_lower_case = 1;
ret = dowild((const uchar*)pattern, (const uchar*)text);
force_lower_case = 0;
return ret;
return dowild((const uchar*)pattern, (const uchar*)text,
flags & FNM_CASEFOLD ? 1 :0);
}
3 changes: 1 addition & 2 deletions wildmatch.h
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
/* wildmatch.h */

int wildmatch(const char *pattern, const char *text);
int iwildmatch(const char *pattern, const char *text);
int wildmatch(const char *pattern, const char *text, int flags);

0 comments on commit 9b4edc0

Please sign in to comment.