Skip to content
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

sprintf() is now optimized by the compiler #1697

Closed
arokettu opened this issue Sep 25, 2024 · 4 comments
Closed

sprintf() is now optimized by the compiler #1697

arokettu opened this issue Sep 25, 2024 · 4 comments

Comments

@arokettu
Copy link

See: php/php-src#14546

Therefore is should be added to FunctionHelper::SPECIAL_FUNCTIONS

@bkdotcom
Copy link
Contributor

bkdotcom commented Oct 2, 2024

has sprintf's function signature changed?
Is it now a language construct (like empty()) vs a function?

what makes it "special" ?

@arokettu
Copy link
Author

arokettu commented Oct 4, 2024

@bkdotcom FunctionHelper::SPECIAL_FUNCTIONS is used by SlevomatCodingStandard.Namespaces.FullyQualifiedGlobalFunctions to indicate functions that can be optimized by compiler when fully qualified, at least that's what the code tells me

@kukulich
Copy link
Contributor

kukulich commented Feb 1, 2025

Implemented in dd05cd6

@arokettu
Copy link
Author

arokettu commented Feb 3, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants