mirror of
https://github.com/php/php-src.git
synced 2026-03-30 12:13:02 +02:00
08b2ab22f4
* Include the source location in Closure names This change makes stack traces involving Closures, especially multiple different Closures, much more useful, because it's more easily visible *which* closure was called for a given stack frame. The implementation is similar to that of anonymous classes which already include the file name and line number within their generated classname. * Update scripts/dev/bless_tests.php for closure naming * Adjust existing tests for closure naming * Adjust tests for closure naming that were not caught locally * Drop the namespace from closure names This is redundant with the included filename. * Include filename and line number as separate keys in Closure debug info * Fix test * Fix test * Include the surrounding class and function name in closure names * Fix test * Relax test expecations * Fix tests after merge * NEWS / UPGRADING
34 lines
573 B
PHP
34 lines
573 B
PHP
--TEST--
|
|
Bug #75290 (debug info of Closures of internal functions contain garbage argument names)
|
|
--FILE--
|
|
<?php
|
|
|
|
var_dump((new ReflectionFunction('sin'))->getClosure());
|
|
|
|
var_dump(function ($someThing) {});
|
|
|
|
?>
|
|
--EXPECTF--
|
|
object(Closure)#%d (2) {
|
|
["function"]=>
|
|
string(3) "sin"
|
|
["parameter"]=>
|
|
array(1) {
|
|
["$num"]=>
|
|
string(10) "<required>"
|
|
}
|
|
}
|
|
object(Closure)#%d (4) {
|
|
["name"]=>
|
|
string(%d) "{closure:%s:%d}"
|
|
["file"]=>
|
|
string(%d) "%s"
|
|
["line"]=>
|
|
int(%d)
|
|
["parameter"]=>
|
|
array(1) {
|
|
["$someThing"]=>
|
|
string(10) "<required>"
|
|
}
|
|
}
|