-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Return signatures of Autoloader's loaders should be void #7747
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
a841c6e
to
14d7221
Compare
@paulbalandan I don't see the benefit. I don't think this is a bug. I think adding |
The signature is in the Parameters section:
Okay, I'll send to |
14d7221
to
31b79b1
Compare
@paulbalandan Okay, the callback return type should be |
31b79b1
to
f075b7c
Compare
Description
Ref: https://www.php.net/manual/en/function.spl-autoload-register.php
As a result of this change, this may be a potential BC break. I added this to the changelog. Also marked
@internal
those 2 callbacks.Checklist: