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

New SAPI reason #31

Open
Danack opened this issue Jul 9, 2022 · 0 comments
Open

New SAPI reason #31

Danack opened this issue Jul 9, 2022 · 0 comments

Comments

@Danack
Copy link
Owner

Danack commented Jul 9, 2022

Ticks is dumb, and I hate the way Github cross links stuff.

https://externals.io/message/114368

To cross post a comment from the PR, in
php/php-src#6967 (comment) someone
mentions that they intercept and rewrite all included files to add
declare(ticks=N) in order to perform memory usage / performance monitoring.

In the same thread someone points out that there is a WordPress proflinig
plugin using the same approach:
php/php-src#6967 (comment)

I think I'm going to withdraw this RFC. Keeping this functionality is not a
particularly large burden on us, and some people are getting some use out
of it, even if those uses are rather ... adventurous.

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

No branches or pull requests

1 participant