diff options
author | brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7> | 2008-09-20 17:52:48 +0000 |
---|---|---|
committer | brain <brain@e03df62e-2008-0410-955e-edbf42e46eb7> | 2008-09-20 17:52:48 +0000 |
commit | b8e537eba7a77bc8cbcdd30e2ff1ba79fb7b0569 (patch) | |
tree | db12a22542f932ca10ae975534ffd71fa80d29b2 | |
parent | a2b94d7a8b3fa734cf56164595a23b442da96daf (diff) |
Roll back core matcher to the 1.1 design, which doesnt fail various unit tests
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@10564 e03df62e-2008-0410-955e-edbf42e46eb7
-rw-r--r-- | src/wildcard.cpp | 104 |
1 files changed, 45 insertions, 59 deletions
diff --git a/src/wildcard.cpp b/src/wildcard.cpp index b49ba943a..63a28b8cb 100644 --- a/src/wildcard.cpp +++ b/src/wildcard.cpp @@ -17,70 +17,56 @@ #include "hashcomp.h" #include "inspstring.h" -/* - * Wildcard matching! - * - * Iteration 1) - * Slow, horrible, etc. - * Iteration 2) - * The vastly available 'public domain' one - * Iteration 3) - * ZNC's, thought to be faster than ours, but it turned out that we could do better ;-) - * Iteration 4) - * Largely from work by peavey and myself (w00t) :) - * Iteration 5) - * peavey: Fix glob scan similar to 1.1, but scan ahead on glob in inner loop to retain speedup - * this fixes another case which we forgot to test. Add early return for obvious fail condition. - */ -static bool match_internal(const unsigned char *string, const unsigned char *wild, unsigned const char *map) +static bool match_internal(const unsigned char *str, const unsigned char *mask, unsigned const char *map) { - const unsigned char *s, *m; m = wild; - - if (*string && !*wild) - return false; + unsigned char *cp = NULL, *mp = NULL; + unsigned char* string = (unsigned char*)str; + unsigned char* wild = (unsigned char*)mask; if (!map) map = rfc_case_insensitive_map; - while (*string) - { - if (*wild == '*') - { - while (*wild && *wild == '*') - wild++; - - m = wild; - - if (!*wild) - return true; - else if (*wild != '?') - { - s = string; - while (*s) - { - if ((map[*wild] == map[*s])) - { - string = s; - if (*(wild+1) || !*(s+1)) - wild++; - break; - } - s++; - } - } - } - else if ( (map[*wild] == map[*string]) || (*wild == '?') ) - wild++; - else - wild = m; - - string++; - } - - while (*wild && *wild == '*') - wild++; - - return !*wild; + while ((*string) && (*wild != '*')) + { + if ((map[*wild] != map[*string]) && (*wild != '?')) + { + return 0; + } + wild++; + string++; + } + + while (*string) + { + if (*wild == '*') + { + if (!*++wild) + { + return 1; + } + mp = wild; + cp = string+1; + } + else + if ((map[*wild] == map[*string]) || (*wild == '?')) + { + wild++; + string++; + } + else + { + wild = mp; + string = cp++; + } + + } + + while (*wild == '*') + { + wild++; + } + + return !*wild; } /******************************************************************** |