Zend: Return anonymous closure names in zend_get_callable_name_ex() (php#19011)

This returns the usual `{closure:FILE_NAME/FUNCTION:LINE_NO}` for anonymous functions rather than `Closure::__invoke` this is visible for `is_callable()` and any Engine call that uses `zend_fcall_info_init()` to get the name of the callable.

Related to phpGH-18063.

Author: Girgias

Zend/tests/closures/closure_016.phpt

@@ -42,9 +42,9 @@ Foo::__invoke
 bool(true)
 Foo::__invoke
 bool(true)
-Closure::__invoke
+{closure:foo():9}
 bool(true)
-Closure::__invoke
+{closure:foo():9}
 bool(true)
 Closure::__invoke
 bool(true)

Zend/zend_API.c

@@ -4160,13 +4160,10 @@ ZEND_API zend_string *zend_get_callable_name_ex(zval *callable, zend_object *obj
 			if (ce == zend_ce_closure) {
 				const zend_function *fn = zend_get_closure_method_def(Z_OBJ_P(callable));
 
-				if (fn->common.fn_flags & ZEND_ACC_FAKE_CLOSURE) {
-					if (fn->common.scope) {
-						return zend_create_member_string(fn->common.scope->name, fn->common.function_name);
-					} else {
-						return zend_string_copy(fn->common.function_name);
-					}
+				if ((fn->common.fn_flags & ZEND_ACC_FAKE_CLOSURE) && fn->common.scope) {
+					return zend_create_member_string(fn->common.scope->name, fn->common.function_name);
 				}
+				return zend_string_copy(fn->common.function_name);
 			}
 
 			return zend_string_concat2(

tests/output/ob_013.phpt

@@ -57,11 +57,11 @@ Array
     [5] => E::f
     [6] => E::g
     [7] => E::__invoke
-    [8] => Closure::__invoke
+    [8] => {closure:%s:%d}
 )
 Array
 (
-    [name] => Closure::__invoke
+    [name] => {closure:%s:%d}
     [type] => 1
     [flags] => 20593
     [level] => 8
@@ -161,7 +161,7 @@ Array
 
     [8] => Array
         (
-            [name] => Closure::__invoke
+            [name] => {closure:%s:%d}
             [type] => 1
             [flags] => 20593
             [level] => 8