Hi,
Just a few minor nitpicks:
+static unsigned char tolower_map[256] = {
This should be const.
+#define zend_tolower_ascii(c) (tolower_map[(c)])
I suggest casting to unsigned char to prevent overflows and negative chars
problems (128-255):
#define zend_tolower_ascii(c) (tolower_map[(unsigned char)(c)])
+ *result++ = zend_tolower_ascii((int)*str++);
The cast to int is not necessary. Same comment applies for most other calls.
And thanks for fixing this ancient bug :)
Nuno
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php