diff options
| author | Sterling Hughes <sterling@php.net> | 2002-10-04 20:44:19 +0000 |
|---|---|---|
| committer | Sterling Hughes <sterling@php.net> | 2002-10-04 20:44:19 +0000 |
| commit | df837e67c7ee99bb022211c079921f497cacaed0 (patch) | |
| tree | 1c029b6456252099660b8212f7e1aab672368cce /ext/standard/string.c | |
| parent | 1fe42e10b7ec02da9a16c478f28e1992e5685f0a (diff) | |
| download | php-git-df837e67c7ee99bb022211c079921f497cacaed0.tar.gz | |
have implode use the smart_str_*() functions, this should speed things up
quite a bit...
Diffstat (limited to 'ext/standard/string.c')
| -rw-r--r-- | ext/standard/string.c | 44 |
1 files changed, 15 insertions, 29 deletions
diff --git a/ext/standard/string.c b/ext/standard/string.c index a8114f7424..a29c9b52a6 100644 --- a/ext/standard/string.c +++ b/ext/standard/string.c @@ -818,42 +818,28 @@ PHP_FUNCTION(explode) */ PHPAPI void php_implode(zval *delim, zval *arr, zval *return_value) { - zval **tmp; - char *tmp_str; - int len = 0, count = 0, target = 0; - HashPosition pos; + zval **tmp; + HashPosition pos; + smart_str implstr = {0}; + int numelems, i = 0; + + numelems = zend_hash_num_elements(Z_ARRVAL_P(arr)); - /* convert everything to strings, and calculate length */ zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(arr), &pos); - while (zend_hash_get_current_data_ex(Z_ARRVAL_P(arr), (void **) &tmp, &pos) == SUCCESS) { + while (zend_hash_get_current_data_ex(Z_ARRVAL_P(arr), + (void **) &tmp, + &pos) == SUCCESS) { convert_to_string_ex(tmp); - len += Z_STRLEN_PP(tmp); - if (count > 0) { - len += Z_STRLEN_P(delim); - } - - count++; - zend_hash_move_forward_ex(Z_ARRVAL_P(arr), &pos); - } - /* do it */ - tmp_str = (char *) emalloc(len + 1); - tmp_str[0] = 0; - - zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(arr), &pos); - while (zend_hash_get_current_data_ex(Z_ARRVAL_P(arr), (void **) &tmp, &pos) == SUCCESS) { - count--; - memcpy(tmp_str + target, Z_STRVAL_PP(tmp), Z_STRLEN_PP(tmp)); - target += Z_STRLEN_PP(tmp); - if (count > 0) { - memcpy(tmp_str + target, Z_STRVAL_P(delim), Z_STRLEN_P(delim)); - target += Z_STRLEN_P(delim); + smart_str_appendl(&implstr, Z_STRVAL_PP(tmp), Z_STRLEN_PP(tmp)); + if (++i != numelems) { + smart_str_appendl(&implstr, Z_STRVAL_P(delim), Z_STRLEN_P(delim)); } zend_hash_move_forward_ex(Z_ARRVAL_P(arr), &pos); } - tmp_str[len] = 0; - - RETURN_STRINGL(tmp_str, len, 0); + smart_str_0(&implstr); + + RETURN_STRINGL(implstr.c, implstr.len, 0); } /* }}} */ |