-
Notifications
You must be signed in to change notification settings - Fork 147
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
Remove non php7 code from ext/php7 folder #1162
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LTGM! Thanks for all the deletions @labbati! 🥇
TSRMLS_FETCH(); | ||
php_log_err((char *)message TSRMLS_CC); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In another followup PR, we could remove the TSRMLS_*
references for PHP 7 and PHP 8. Then this would become:
TSRMLS_FETCH(); | |
php_log_err((char *)message TSRMLS_CC); | |
php_log_err((char *)message); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am leaving this for a separate PR as I want to discuss this further.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even better. Thanks @labbati! 👍
Description
Get rid of all non php7 code in
ext/php7
Readiness checklist
[ ] (only for Members) Changelog has been added to the release document.[ ] Tests added for this feature/bug.Reviewer checklist