code de-duplication in ReflectionType::__toString and ReflectionNamedType::getName

This code duplication introduce an inconsistency in displayed type name
- bool (reflection) vs boolean
- int (reflection) vs integer

And reflection already use zend_get_type_by_const in other methods...

Inconsistenty is kept for BC reason.
Could be fixed in 8.0

1 files changed, 7 insertions, 7 deletions

diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index 19975d29bc..badbf8eb88 100644
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -2904,15 +2904,15 @@ static zend_string *reflection_type_name(type_reference *param) {
 		return zend_string_copy(ZEND_TYPE_NAME(param->arg_info->type));
 	}
 	switch (ZEND_TYPE_CODE(param->arg_info->type)) {
-		case IS_ARRAY:    return zend_string_init("array", sizeof("array") - 1, 0);
-		case IS_CALLABLE: return zend_string_init("callable", sizeof("callable") - 1, 0);
-		case IS_STRING:   return zend_string_init("string", sizeof("string") - 1, 0);
+		/* keep this for BC, bool vs boolean, int vs integer */
 		case _IS_BOOL:    return zend_string_init("bool", sizeof("bool") - 1, 0);
 		case IS_LONG:     return zend_string_init("int", sizeof("int") - 1, 0);
-		case IS_DOUBLE:   return zend_string_init("float", sizeof("float") - 1, 0);
-		case IS_VOID:     return zend_string_init("void", sizeof("void") - 1, 0);
-		case IS_ITERABLE: return zend_string_init("iterable", sizeof("iterable") - 1, 0);
-		EMPTY_SWITCH_DEFAULT_CASE()
+		/* use zend API for other types */
+		default:
+			{
+			char *name = zend_get_type_by_const(ZEND_TYPE_CODE(param->arg_info->type));
+			return zend_string_init(name, strlen(name), 0);
+			}
 	}
 }
 /* }}} */