1
0
mirror of https://github.com/php/php-src.git synced 2026-03-24 08:12:21 +01:00
Files
Tim Düsterhus 08b2ab22f4 Include the source location in Closure names (#13550)
* 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
2024-04-12 18:21:13 +02:00

151 lines
2.5 KiB
PHP

--TEST--
ReflectionClass::newInstance[Args]
--FILE--
<?php
function test($class)
{
echo "====>$class\n";
try
{
$ref = new ReflectionClass($class);
}
catch (ReflectionException $e)
{
var_dump($e->getMessage());
return; // only here
}
echo "====>newInstance()\n";
try
{
var_dump($ref->newInstance());
}
catch (ReflectionException $e)
{
var_dump($e->getMessage());
}
catch (Throwable $e)
{
echo "Exception: " . $e->getMessage() . "\n";
}
echo "====>newInstance(25)\n";
try
{
var_dump($ref->newInstance(25));
}
catch (ReflectionException $e)
{
var_dump($e->getMessage());
}
echo "====>newInstance(25, 42)\n";
try
{
var_dump($ref->newInstance(25, 42));
}
catch (ReflectionException $e)
{
var_dump($e->getMessage());
}
echo "\n";
}
spl_autoload_register(function ($class) {
echo __FUNCTION__ . "($class)\n";
});
test('Class_does_not_exist');
Class NoCtor
{
}
test('NoCtor');
Class WithCtor
{
function __construct()
{
echo __METHOD__ . "()\n";
var_dump(func_get_args());
}
}
test('WithCtor');
Class WithCtorWithArgs
{
function __construct($arg)
{
echo __METHOD__ . "($arg)\n";
var_dump(func_get_args());
}
}
test('WithCtorWithArgs');
?>
--EXPECTF--
====>Class_does_not_exist
{closure:%s:%d}(Class_does_not_exist)
string(43) "Class "Class_does_not_exist" does not exist"
====>NoCtor
====>newInstance()
object(NoCtor)#%d (0) {
}
====>newInstance(25)
string(86) "Class NoCtor does not have a constructor, so you cannot pass any constructor arguments"
====>newInstance(25, 42)
string(86) "Class NoCtor does not have a constructor, so you cannot pass any constructor arguments"
====>WithCtor
====>newInstance()
WithCtor::__construct()
array(0) {
}
object(WithCtor)#%d (0) {
}
====>newInstance(25)
WithCtor::__construct()
array(1) {
[0]=>
int(25)
}
object(WithCtor)#%d (0) {
}
====>newInstance(25, 42)
WithCtor::__construct()
array(2) {
[0]=>
int(25)
[1]=>
int(42)
}
object(WithCtor)#%d (0) {
}
====>WithCtorWithArgs
====>newInstance()
Exception: Too few arguments to function WithCtorWithArgs::__construct(), 0 passed and exactly 1 expected
====>newInstance(25)
WithCtorWithArgs::__construct(25)
array(1) {
[0]=>
int(25)
}
object(WithCtorWithArgs)#%d (0) {
}
====>newInstance(25, 42)
WithCtorWithArgs::__construct(25)
array(2) {
[0]=>
int(25)
[1]=>
int(42)
}
object(WithCtorWithArgs)#%d (0) {
}