diff options
author | Xinchen Hui <laruence@php.net> | 2011-08-08 06:49:17 +0000 |
---|---|---|
committer | Xinchen Hui <laruence@php.net> | 2011-08-08 06:49:17 +0000 |
commit | a7852bcb00f1911b97b22facf572181be33d58c2 (patch) | |
tree | 6ade292c7b8fffd9d96c86cb3e3758031f5afba6 /ext/standard/info.c | |
parent | ed7b3e703e7807e2c9e2d30bda26e530e93e0f45 (diff) | |
download | php-git-a7852bcb00f1911b97b22facf572181be33d58c2.tar.gz |
Make codes more clearly, and make the static analyzer silent..
Diffstat (limited to 'ext/standard/info.c')
-rw-r--r-- | ext/standard/info.c | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/ext/standard/info.c b/ext/standard/info.c index 2bee7e9803..f57e4afab3 100644 --- a/ext/standard/info.c +++ b/ext/standard/info.c @@ -245,22 +245,22 @@ static void php_print_gpcse_array(char *name, uint name_length TSRMLS_DC) } } else { tmp2 = **tmp; - switch (Z_TYPE_PP(tmp)) { - default: - tmp = NULL; - zval_copy_ctor(&tmp2); - convert_to_string(&tmp2); - case IS_STRING: - if (!sapi_module.phpinfo_as_text) { - if (Z_STRLEN(tmp2) == 0) { - php_info_print("<i>no value</i>"); - } else { - php_info_print_html_esc(Z_STRVAL(tmp2), Z_STRLEN(tmp2)); - } - } else { - php_info_print(Z_STRVAL(tmp2)); - } + if (Z_TYPE_PP(tmp) != IS_STRING) { + tmp = NULL; + zval_copy_ctor(&tmp2); + convert_to_string(&tmp2); } + + if (!sapi_module.phpinfo_as_text) { + if (Z_STRLEN(tmp2) == 0) { + php_info_print("<i>no value</i>"); + } else { + php_info_print_html_esc(Z_STRVAL(tmp2), Z_STRLEN(tmp2)); + } + } else { + php_info_print(Z_STRVAL(tmp2)); + } + if (!tmp) { zval_dtor(&tmp2); } |