diff options
| author | Dmitry Stogov <dmitry@zend.com> | 2018-06-01 11:58:57 +0300 |
|---|---|---|
| committer | Dmitry Stogov <dmitry@zend.com> | 2018-06-01 11:58:57 +0300 |
| commit | 0b90cf85a6ace22abd0f9f86d5015a5b6db92387 (patch) | |
| tree | e01be0712ad1d7dbcd61c81ce03b9544a69e4972 /ext/mbstring/mbstring.c | |
| parent | 9e0f131d2b0e8bf98859c12104a7b0cd8951c135 (diff) | |
| download | php-git-0b90cf85a6ace22abd0f9f86d5015a5b6db92387.tar.gz | |
Removed "dead" code (zend_hash_update() never fails)
Diffstat (limited to 'ext/mbstring/mbstring.c')
| -rw-r--r-- | ext/mbstring/mbstring.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/ext/mbstring/mbstring.c b/ext/mbstring/mbstring.c index 8ed185c76a..b17b8f8f8e 100644 --- a/ext/mbstring/mbstring.c +++ b/ext/mbstring/mbstring.c @@ -1608,7 +1608,6 @@ ZEND_TSRMLS_CACHE_UPDATE(); zend_function *func, *orig; const struct mb_overload_def *p; zend_string *str; - void *ret; p = &(mb_ovld[0]); while (p->type > 0) { @@ -1628,13 +1627,8 @@ ZEND_TSRMLS_CACHE_UPDATE(); function_add_ref(orig); str = zend_string_init_interned(p->orig_func, strlen(p->orig_func), 1); - ret = zend_hash_update_mem(CG(function_table), str, func, sizeof(zend_internal_function)); + zend_hash_update_mem(CG(function_table), str, func, sizeof(zend_internal_function)); zend_string_release_ex(str, 1); - if (ret == NULL) { - php_error_docref("ref.mbstring", E_WARNING, "mbstring couldn't replace function %s.", p->orig_func); - return FAILURE; - } - function_add_ref(func); } } |