Dead code or autoloading mechanism?

[holtkamp]

Maybe I am missing the point, but what is the functional justification of this snippet:

tracy/src/Tracy/Debugger/Debugger.php

Lines 202 to 203 in eb2e772

	array_map('class_exists', [Bar::class, BlueScreen::class, DefaultBarPanel::class, Dumper::class,
	FireLogger::class, Helpers::class, Logger::class, ]);

Is it to ensure these classes are "autoloaded"? If so, what would be the point of that?

[jelen07]

I think that the primary aim of this code is that ensures, that Tracy will work properly. It's better to know earlier, that Tracy won't work in Debugger::enable() method, than later - ie. logging and so on.

[dg]

This is because the errors that occurs in the autoloader.

For questions please use the forum https://forum.nette.org/en/f46-tracy

[holtkamp]

This is because the errors that occurs in the autoloader.

ok, so it is intended to "capture" any error that might occur during autoloading. But it does not ensure that class exists... class_exists('unexistingClass'); simply returns false...

For questions please use the forum https://forum.nette.org/en/f46-tracy

Ok, sorry. Thought this was more a "issue" than a question 😉

Closing for now, but I think the code does not do what it suggests it does... 🤔

[dg]

It is just fix for very specific problem. It can be replaced with require_once ....