moriyoshi Thu Jul 24 13:46:50 2008 UTC Added files: (Branch: PHP_5_3) /php-src/ext/mbstring/tests mb_strstr.phpt
Modified files: /php-src/ext/mbstring mbstring.c mbstring.h php_unicode.c php_unicode.h Log: - MFH: Fixed warnings.
http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/mbstring.c?r1=1.224.2.22.2.25.2.20&r2=1.224.2.22.2.25.2.21&diff_format=u Index: php-src/ext/mbstring/mbstring.c diff -u php-src/ext/mbstring/mbstring.c:1.224.2.22.2.25.2.20 php-src/ext/mbstring/mbstring.c:1.224.2.22.2.25.2.21 --- php-src/ext/mbstring/mbstring.c:1.224.2.22.2.25.2.20 Thu Jul 24 13:01:49 2008 +++ php-src/ext/mbstring/mbstring.c Thu Jul 24 13:46:49 2008 @@ -17,7 +17,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: mbstring.c,v 1.224.2.22.2.25.2.20 2008/07/24 13:01:49 moriyoshi Exp $ */ +/* $Id: mbstring.c,v 1.224.2.22.2.25.2.21 2008/07/24 13:46:49 moriyoshi Exp $ */ /* * PHP 4 Multibyte String module "mbstring" @@ -2225,17 +2225,17 @@ { int n; long offset; - char *old_haystack, *old_needle; + mbfl_string haystack, needle; char *from_encoding = (char*)mbfl_no2preferred_mime_name(MBSTRG(current_internal_encoding)); - int old_haystack_len, old_needle_len, from_encoding_len; + int from_encoding_len; n = -1; offset = 0; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|ls", &old_haystack, &old_haystack_len, &old_needle, &old_needle_len, &offset, &from_encoding, &from_encoding_len ) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|ls", (char **)&haystack.val, (int *)&haystack.len, (char **)needle.val, (int *)&needle.len, &offset, &from_encoding, &from_encoding_len) == FAILURE) { RETURN_FALSE; } - n = php_mb_stripos(0, old_haystack, old_haystack_len, old_needle, old_needle_len, offset, from_encoding TSRMLS_CC); + n = php_mb_stripos(0, (char *)haystack.val, haystack.len, (char *)needle.val, needle.len, offset, from_encoding TSRMLS_CC); if (n >= 0) { RETVAL_LONG(n); @@ -2251,21 +2251,21 @@ { int n; long offset; - char *old_haystack, *old_needle; - char *from_encoding = (char*)mbfl_no2preferred_mime_name(MBSTRG(current_internal_encoding)); - int old_haystack_len, old_needle_len, from_encoding_len; + mbfl_string haystack, needle; + const char *from_encoding = mbfl_no2preferred_mime_name(MBSTRG(current_internal_encoding)); + int from_encoding_len; n = -1; offset = 0; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|ls", &old_haystack, &old_haystack_len, &old_needle, &old_needle_len, &offset, &from_encoding, &from_encoding_len ) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|ls", (char **)&haystack.val, (int *)&haystack.len, (char **)&needle.val, (int *)&needle.len, &offset, &from_encoding, &from_encoding_len) == FAILURE) { RETURN_FALSE; } - if(offset > old_haystack_len){ + if (offset > haystack.len) { RETURN_FALSE; } - n = php_mb_stripos(1, old_haystack, old_haystack_len, old_needle, old_needle_len, offset, from_encoding TSRMLS_CC); + n = php_mb_stripos(1, (char *)haystack.val, haystack.len, (char *)needle.val, needle.len, offset, from_encoding TSRMLS_CC); if (n >= 0) { RETVAL_LONG(n); @@ -2292,7 +2292,7 @@ needle.no_language = MBSTRG(current_language); needle.no_encoding = MBSTRG(current_internal_encoding); - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|bs", (char **)&haystack.val, &haystack.len, (char **)&needle.val, &needle.len, &part, &enc_name, &enc_name_len) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|bs", (char **)&haystack.val, (int *)&haystack.len, (char **)&needle.val, (int *)&needle.len, &part, &enc_name, &enc_name_len) == FAILURE) { RETURN_FALSE; } @@ -2398,9 +2398,9 @@ PHP_FUNCTION(mb_stristr) { zend_bool part = 0; - int n, from_encoding_len, len, mblen; + unsigned int n, from_encoding_len, len, mblen; mbfl_string haystack, needle, result, *ret = NULL; - char *from_encoding = (char*)mbfl_no2preferred_mime_name(MBSTRG(current_internal_encoding)); + const char *from_encoding = mbfl_no2preferred_mime_name(MBSTRG(current_internal_encoding)); mbfl_string_init(&haystack); mbfl_string_init(&needle); haystack.no_language = MBSTRG(current_language); @@ -2413,7 +2413,7 @@ RETURN_FALSE; } - if(!needle.len){ + if (!needle.len) { php_error_docref(NULL TSRMLS_CC, E_WARNING,"Empty delimiter."); RETURN_FALSE; } @@ -2424,7 +2424,7 @@ RETURN_FALSE; } - n = php_mb_stripos(0, haystack.val, haystack.len, needle.val, needle.len, 0, from_encoding TSRMLS_CC); + n = php_mb_stripos(0, (char *)haystack.val, haystack.len, (char *)needle.val, needle.len, 0, from_encoding TSRMLS_CC); if (n <0) { RETURN_FALSE; @@ -2477,7 +2477,7 @@ RETURN_FALSE; } - n = php_mb_stripos(1, haystack.val, haystack.len, needle.val, needle.len, 0, from_encoding TSRMLS_CC); + n = php_mb_stripos(1, (char *)haystack.val, haystack.len, (char *)needle.val, needle.len, 0, from_encoding TSRMLS_CC); if (n <0) { RETURN_FALSE; @@ -2624,19 +2624,19 @@ PHP_FUNCTION(mb_strcut) { size_t argc = ZEND_NUM_ARGS(); - char *str, *encoding; + char *encoding; long from, len; - int str_len, encoding_len; + int encoding_len; mbfl_string string, result, *ret; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sl|ls", &str, &str_len, &from, &len, &encoding, &encoding_len) == FAILURE) { - return; - } - mbfl_string_init(&string); string.no_language = MBSTRG(current_language); string.no_encoding = MBSTRG(current_internal_encoding); + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sl|ls", (char **)&string.val, (int **)&string.len, &from, &len, &encoding, &encoding_len) == FAILURE) { + return; + } + if (argc == 4) { string.no_encoding = mbfl_name2no_encoding(encoding); if (string.no_encoding == mbfl_no_encoding_invalid) { @@ -2645,18 +2645,15 @@ } } - string.val = str; - string.len = str_len; - if (argc < 3) { - len = str_len; + len = string.len; } /* if "from" position is negative, count start position from the end * of the string */ if (from < 0) { - from = str_len + from; + from = string.len + from; if (from < 0) { from = 0; } @@ -2666,18 +2663,18 @@ * needed to stop that many chars from the end of the string */ if (len < 0) { - len = (str_len - from) + len; + len = (string.len - from) + len; if (len < 0) { len = 0; } } - if (from > str_len) { + if (from > string.len) { RETURN_FALSE; } - if (((unsigned) from + (unsigned) len) > str_len) { - len = str_len - from; + if (((unsigned) from + (unsigned) len) > string.len) { + len = string.len - from; } ret = mbfl_strcut(&string, &result, from, len); @@ -2685,7 +2682,7 @@ RETURN_FALSE; } - RETURN_STRINGL(ret->val, ret->len, 0); /* the string is already strdup()'ed */ + RETURN_STRINGL((char *)ret->val, ret->len, 0); /* the string is already strdup()'ed */ } /* }}} */ @@ -2783,7 +2780,7 @@ /* }}} */ /* {{{ MBSTRING_API char *php_mb_convert_encoding() */ -MBSTRING_API char * php_mb_convert_encoding(char *input, size_t length, char *_to_encoding, char *_from_encodings, size_t *output_len TSRMLS_DC) +MBSTRING_API char * php_mb_convert_encoding(const char *input, size_t length, const char *_to_encoding, const char *_from_encodings, size_t *output_len TSRMLS_DC) { mbfl_string string, result, *ret; enum mbfl_no_encoding from_encoding, to_encoding; @@ -3596,18 +3593,18 @@ } else { zval_dtor(*hash_entry); } - ZVAL_STRINGL(*hash_entry, ret->val, ret->len, 0); - } + ZVAL_STRINGL(*hash_entry, (char *)ret->val, ret->len, 0); } } } - } else if (Z_TYPE_PP(var) == IS_STRING) { - string.val = (unsigned char *)Z_STRVAL_PP(var); - string.len = Z_STRLEN_PP(var); - ret = mbfl_buffer_converter_feed_result(convd, &string, &result); - if (ret != NULL) { - zval_dtor(*var); - ZVAL_STRINGL(*var, ret->val, ret->len, 0); + } + } else if (Z_TYPE_PP(var) == IS_STRING) { + string.val = (unsigned char *)Z_STRVAL_PP(var); + string.len = Z_STRLEN_PP(var); + ret = mbfl_buffer_converter_feed_result(convd, &string, &result); + if (ret != NULL) { + zval_dtor(*var); + ZVAL_STRINGL(*var, (char *)ret->val, ret->len, 0); } } } @@ -3938,15 +3935,15 @@ PHP_FUNCTION(mb_send_mail) { int n; - char *to=NULL; + char *to = NULL; int to_len; - char *message=NULL; + char *message = NULL; int message_len; - char *headers=NULL; + char *headers = NULL; int headers_len; - char *subject=NULL; + char *subject = NULL; int subject_len; - char *extra_cmd=NULL; + char *extra_cmd = NULL; int extra_cmd_len; int i; char *to_r = NULL; @@ -3956,7 +3953,7 @@ int cnt_trans_enc:1; } suppressed_hdrs = { 0, 0 }; - char *message_buf=NULL, *subject_buf=NULL, *p; + char *message_buf = NULL, *subject_buf = NULL, *p; mbfl_string orig_str, conv_str; mbfl_string *pstr; /* pointer to mbfl string for return value */ enum mbfl_no_encoding @@ -4123,8 +4120,8 @@ /* message body */ if (message != NULL) { orig_str.no_language = MBSTRG(current_language); - orig_str.val = message; - orig_str.len = message_len; + orig_str.val = (unsigned char *)message; + orig_str.len = (unsigned int)message_len; orig_str.no_encoding = MBSTRG(current_internal_encoding); if (orig_str.no_encoding == mbfl_no_encoding_invalid @@ -4495,7 +4492,7 @@ if (ret != NULL) { MBSTRG(illegalchars) += illegalchars; - if (illegalchars == 0 && strncmp(string.val, ret->val, string.len) == 0) { + if (illegalchars == 0 && strncmp((char *)string.val, (char *)ret->val, string.len) == 0) { efree(ret->val); RETURN_TRUE; } else { @@ -4650,7 +4647,7 @@ string.no_language = MBSTRG(current_language); for (i=0; i<num; i++){ - string.val = (char*)str[i]; + string.val = (unsigned char *)str[i]; string.len = len[i]; /* initialize converter */ @@ -4665,8 +4662,8 @@ ret = mbfl_buffer_converter_feed_result(convd, &string, &result); if (ret != NULL) { efree(str[i]); - str[i] = ret->val; - len[i] = ret->len; + str[i] = (char *)ret->val; + len[i] = (int)ret->len; } MBSTRG(illegalchars) += mbfl_buffer_illegalchars(convd); @@ -4757,8 +4754,7 @@ /* {{{ MBSTRING_API int php_mb_stripos() */ - -MBSTRING_API int php_mb_stripos(int mode, char *old_haystack, int old_haystack_len, char *old_needle, int old_needle_len, long offset, char *from_encoding TSRMLS_DC) +MBSTRING_API int php_mb_stripos(int mode, const char *old_haystack, unsigned int old_haystack_len, const char *old_needle, unsigned int old_needle_len, unsigned int offset, const char *from_encoding TSRMLS_DC) { int n; mbfl_string haystack, needle; @@ -4772,7 +4768,9 @@ needle.no_encoding = MBSTRG(current_internal_encoding); do { - haystack.val = php_unicode_convert_case(PHP_UNICODE_CASE_UPPER, old_haystack, (size_t) old_haystack_len, &haystack.len, from_encoding TSRMLS_CC); + size_t len = 0; + haystack.val = (unsigned char *)php_unicode_convert_case(PHP_UNICODE_CASE_UPPER, (char *)old_haystack, old_haystack_len, &len, from_encoding TSRMLS_CC); + haystack.len = len; if (!haystack.val) { break; @@ -4782,7 +4780,8 @@ break; } - needle.val = php_unicode_convert_case(PHP_UNICODE_CASE_UPPER, old_needle, (size_t) old_needle_len, &needle.len, from_encoding TSRMLS_CC); + needle.val = (unsigned char *)php_unicode_convert_case(PHP_UNICODE_CASE_UPPER, (char *)old_needle, old_needle_len, &len, from_encoding TSRMLS_CC); + needle.len = len; if (!needle.val) { break; @@ -4798,7 +4797,7 @@ break; } - if (offset < 0 || (unsigned long)offset > haystack.len) { + if (offset < 0 || offset > haystack.len) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Offset not contained in string."); break; } @@ -4806,15 +4805,15 @@ n = mbfl_strpos(&haystack, &needle, offset, mode); } while(0); - if(haystack.val){ + if (haystack.val) { efree(haystack.val); } - if(needle.val){ + if (needle.val) { efree(needle.val); } - return n; + return n; } /* }}} */ http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/mbstring.h?r1=1.66.2.4.2.5.2.4&r2=1.66.2.4.2.5.2.5&diff_format=u Index: php-src/ext/mbstring/mbstring.h diff -u php-src/ext/mbstring/mbstring.h:1.66.2.4.2.5.2.4 php-src/ext/mbstring/mbstring.h:1.66.2.4.2.5.2.5 --- php-src/ext/mbstring/mbstring.h:1.66.2.4.2.5.2.4 Thu Jul 24 13:01:50 2008 +++ php-src/ext/mbstring/mbstring.h Thu Jul 24 13:46:49 2008 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: mbstring.h,v 1.66.2.4.2.5.2.4 2008/07/24 13:01:50 moriyoshi Exp $ */ +/* $Id: mbstring.h,v 1.66.2.4.2.5.2.5 2008/07/24 13:46:49 moriyoshi Exp $ */ /* * PHP 4 Multibyte String module "mbstring" (currently only for Japanese) @@ -137,9 +137,9 @@ size_t nbytes TSRMLS_DC); MBSTRING_API char *php_mb_strrchr(const char *s, char c TSRMLS_DC); -MBSTRING_API char * php_mb_convert_encoding(char *input, size_t length, - char *_to_encoding, - char *_from_encodings, +MBSTRING_API char * php_mb_convert_encoding(const char *input, size_t length, + const char *_to_encoding, + const char *_from_encodings, size_t *output_len TSRMLS_DC); MBSTRING_API int php_mb_check_encoding_list(const char *encoding_list TSRMLS_DC); @@ -158,7 +158,7 @@ MBSTRING_API int php_mb_gpc_encoding_detector(char **arg_string, int *arg_length, int num, char *arg_list TSRMLS_DC); -MBSTRING_API int php_mb_stripos(int mode, char *old_haystack, int old_haystack_len, char *old_needle, int old_needle_len, long offset, char *from_encoding TSRMLS_DC); +MBSTRING_API int php_mb_stripos(int mode, const char *old_haystack, unsigned int old_haystack_len, const char *old_needle, unsigned int old_needle_len, unsigned int offset, const char *from_encoding TSRMLS_DC); ZEND_BEGIN_MODULE_GLOBALS(mbstring) enum mbfl_no_language language; http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/php_unicode.c?r1=1.11.2.2.2.2.2.2&r2=1.11.2.2.2.2.2.3&diff_format=u Index: php-src/ext/mbstring/php_unicode.c diff -u php-src/ext/mbstring/php_unicode.c:1.11.2.2.2.2.2.2 php-src/ext/mbstring/php_unicode.c:1.11.2.2.2.2.2.3 --- php-src/ext/mbstring/php_unicode.c:1.11.2.2.2.2.2.2 Sat Feb 16 12:01:43 2008 +++ php-src/ext/mbstring/php_unicode.c Thu Jul 24 13:46:49 2008 @@ -267,8 +267,8 @@ ((unsigned char*)(ptr))[3] = (v ) & 0xff;\ } -MBSTRING_API char *php_unicode_convert_case(int case_mode, char *srcstr, size_t srclen, size_t *ret_len, - char *src_encoding TSRMLS_DC) +MBSTRING_API char *php_unicode_convert_case(int case_mode, const char *srcstr, size_t srclen, size_t *ret_len, + const char *src_encoding TSRMLS_DC) { char *unicode, *newstr; size_t unicode_len; http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/php_unicode.h?r1=1.8.2.1.2.1.2.1&r2=1.8.2.1.2.1.2.2&diff_format=u Index: php-src/ext/mbstring/php_unicode.h diff -u php-src/ext/mbstring/php_unicode.h:1.8.2.1.2.1.2.1 php-src/ext/mbstring/php_unicode.h:1.8.2.1.2.1.2.2 --- php-src/ext/mbstring/php_unicode.h:1.8.2.1.2.1.2.1 Mon Dec 31 07:17:09 2007 +++ php-src/ext/mbstring/php_unicode.h Thu Jul 24 13:46:49 2008 @@ -103,8 +103,8 @@ MBSTRING_API int php_unicode_is_prop(unsigned long code, unsigned long mask1, unsigned long mask2); -MBSTRING_API char *php_unicode_convert_case(int case_mode, char *srcstr, size_t srclen, size_t *retlen, - char *src_encoding TSRMLS_DC); +MBSTRING_API char *php_unicode_convert_case(int case_mode, const char *srcstr, size_t srclen, size_t *retlen, + const char *src_encoding TSRMLS_DC); #define PHP_UNICODE_CASE_UPPER 0 #define PHP_UNICODE_CASE_LOWER 1 http://cvs.php.net/viewvc.cgi/php-src/ext/mbstring/tests/mb_strstr.phpt?view=markup&rev=1.1 Index: php-src/ext/mbstring/tests/mb_strstr.phpt +++ php-src/ext/mbstring/tests/mb_strstr.phpt
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php