Thanks :)
Nuno

-----Original Message----- From: Stanislav Malyshev
Sent: Sunday, July 15, 2012 5:06 AM
To: php-cvs@lists.php.net
Subject: [PHP-CVS] com php-src: improve code in zend_operators: Zend/zend_operators.c

Commit:    efdb7ce4b01c71f2de1a0de13ffe1a112c7449b0
Author: Stanislav Malyshev <s...@php.net> Sat, 14 Jul 2012 21:06:40 -0700
Parents:   455f3de80267b68fc4d319275c3ca1cc46a44bb3
Branches:  master

Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=efdb7ce4b01c71f2de1a0de13ffe1a112c7449b0

Log:
improve code in zend_operators

Changed paths:
 M  Zend/zend_operators.c


Diff:
diff --git a/Zend/zend_operators.c b/Zend/zend_operators.c
index 8805eb2..93dca94 100644
--- a/Zend/zend_operators.c
+++ b/Zend/zend_operators.c
@@ -42,7 +42,7 @@ static _locale_t current_locale = NULL;

#define TYPE_PAIR(t1,t2) (((t1) << 4) | (t2))

-static unsigned char tolower_map[256] = {
+static const unsigned char tolower_map[256] = {
0x00,0x01,0x02,0x03,0x04,0x05,0x06,0x07,0x08,0x09,0x0a,0x0b,0x0c,0x0d,0x0e,0x0f,
0x10,0x11,0x12,0x13,0x14,0x15,0x16,0x17,0x18,0x19,0x1a,0x1b,0x1c,0x1d,0x1e,0x1f,
0x20,0x21,0x22,0x23,0x24,0x25,0x26,0x27,0x28,0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,
@@ -61,7 +61,7 @@ static unsigned char tolower_map[256] = {
0xf0,0xf1,0xf2,0xf3,0xf4,0xf5,0xf6,0xf7,0xf8,0xf9,0xfa,0xfb,0xfc,0xfd,0xfe,0xff
};

-#define zend_tolower_ascii(c) (tolower_map[(c)])
+#define zend_tolower_ascii(c) (tolower_map[(unsigned char)(c)])

/**
 * Functions using locale lowercase:
@@ -1945,7 +1945,7 @@ ZEND_API char *zend_str_tolower_copy(char *dest, const char *source, unsigned in
 register unsigned char *end = str + length;

 while (str < end) {
- *result++ = zend_tolower_ascii((int)*str++);
+ *result++ = zend_tolower_ascii(*str++);
 }
 *result = '\0';

@@ -1965,7 +1965,7 @@ ZEND_API void zend_str_tolower(char *str, unsigned int length) /* {{{ */
 register unsigned char *end = p + length;

 while (p < end) {
- *p = zend_tolower_ascii((int)*p);
+ *p = zend_tolower_ascii(*p);
 p++;
 }
}
@@ -2014,8 +2014,8 @@ ZEND_API int zend_binary_strcasecmp(const char *s1, uint len1, const char *s2, u

 len = MIN(len1, len2);
 while (len--) {
- c1 = zend_tolower_ascii((int)*(unsigned char *)s1++);
- c2 = zend_tolower_ascii((int)*(unsigned char *)s2++);
+ c1 = zend_tolower_ascii(*(unsigned char *)s1++);
+ c2 = zend_tolower_ascii(*(unsigned char *)s2++);
 if (c1 != c2) {
 return c1 - c2;
 }
@@ -2035,8 +2035,8 @@ ZEND_API int zend_binary_strncasecmp(const char *s1, uint len1, const char *s2,
 }
 len = MIN(length, MIN(len1, len2));
 while (len--) {
- c1 = zend_tolower_ascii((int)*(unsigned char *)s1++);
- c2 = zend_tolower_ascii((int)*(unsigned char *)s2++);
+ c1 = zend_tolower_ascii(*(unsigned char *)s1++);
+ c2 = zend_tolower_ascii(*(unsigned char *)s2++);
 if (c1 != c2) {
 return c1 - c2;
}

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to