summaryrefslogtreecommitdiff
path: root/ext/reflection/php_reflection.c
diff options
context:
space:
mode:
authorc9s <yoanlin93@gmail.com>2015-10-22 20:59:42 +0800
committerXinchen Hui <laruence@gmail.com>2015-10-23 11:19:19 +0800
commitbbaf6daa9db6e6389181b0cc72175cfa023a398c (patch)
tree05f44ee08704bfff6b4a59dbd31051e726e067f1 /ext/reflection/php_reflection.c
parent3dc0bf7428f32908d4c1fe0236e5d0ef23a8eed4 (diff)
downloadphp-git-bbaf6daa9db6e6389181b0cc72175cfa023a398c.tar.gz
Fix boolean conversion warnings
Summary: The compiler complains and raised some warnings about boolean conversion: warning: address of 'ce->constants_table' will always evaluate to 'true' [-Wpointer-bool-conversion] Since the address of 'HashTable' will always evaluate to true. the condition should be removed. The scope is kept for local variables. Platform: OS X 10.11 Compiler: Apple LLVM version 7.0.0 (clang-700.0.72) Target: x86_64-apple-darwin15.0.0 Thread model: posix
Diffstat (limited to 'ext/reflection/php_reflection.c')
-rw-r--r--ext/reflection/php_reflection.c10
1 files changed, 0 insertions, 10 deletions
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index df905bf5cc..6890d0eeaf 100644
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -445,7 +445,6 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
}
/* Constants */
- if (&ce->constants_table) {
string_printf(str, "\n");
count = zend_hash_num_elements(&ce->constants_table);
string_printf(str, "%s - Constants [%d] {\n", indent, count);
@@ -459,10 +458,8 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
} ZEND_HASH_FOREACH_END();
}
string_printf(str, "%s }\n", indent);
- }
/* Static properties */
- if (&ce->properties_info) {
/* counting static properties */
count = zend_hash_num_elements(&ce->properties_info);
if (count > 0) {
@@ -489,10 +486,8 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
} ZEND_HASH_FOREACH_END();
}
string_printf(str, "%s }\n", indent);
- }
/* Static methods */
- if (&ce->function_table) {
/* counting static methods */
count = zend_hash_num_elements(&ce->function_table);
if (count > 0) {
@@ -524,10 +519,8 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
string_printf(str, "\n");
}
string_printf(str, "%s }\n", indent);
- }
/* Default/Implicit properties */
- if (&ce->properties_info) {
count = zend_hash_num_elements(&ce->properties_info) - count_static_props - count_shadow_props;
string_printf(str, "\n%s - Properties [%d] {\n", indent, count);
if (count > 0) {
@@ -540,7 +533,6 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
} ZEND_HASH_FOREACH_END();
}
string_printf(str, "%s }\n", indent);
- }
if (obj && Z_TYPE_P(obj) == IS_OBJECT && Z_OBJ_HT_P(obj)->get_properties) {
string dyn;
@@ -568,7 +560,6 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
}
/* Non static methods */
- if (&ce->function_table) {
count = zend_hash_num_elements(&ce->function_table) - count_static_funcs;
if (count > 0) {
zend_function *mptr;
@@ -617,7 +608,6 @@ static void _class_string(string *str, zend_class_entry *ce, zval *obj, char *in
string_printf(str, "\n%s - Methods [0] {\n", indent);
}
string_printf(str, "%s }\n", indent);
- }
string_printf(str, "%s}\n", indent);
string_free(&sub_indent);