Skip to content

Update opcache return type info for gd functions #4785

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

Closed
wants to merge 1 commit into from

Conversation

TysonAndre
Copy link
Contributor

Many of these now have reflection types,
making the old signatures redundant or inaccurate.
(Some methods also throw Errors instead of returning null/false)

Many of these now have reflection types,
making the old signatures redundant or inaccurate.
(Some methods also throw Errors instead of returning null/false)
@TysonAndre
Copy link
Contributor Author

https://gist.github.com/TysonAndre/922f630d4cdae853580e9faad12a785e contains this and other inconsistencies or redundant annotations that are currently found in zend_func_info.c (with the extensions I have enabled)

https://github.com/phan/phan/blob/master/internal/extract_arg_info.php was the script used to compare the latest reflection types and opcache's zend_func_info.c.

@TysonAndre
Copy link
Contributor Author

The build failures are in tests unrelated to the gd extension and seem sporadic. e.g. php-src is failing on mac OS - Does gc has a small percentage chance of getting triggered on session close? https://dev.azure.com/phpazuredevops/PHP/_build/results?buildId=3136&view=ms.vss-test-web.build-test-results-tab&runId=65728&resultId=108473&paneView=debug

@php-pulls php-pulls closed this in dd61edf Oct 7, 2019
@nikic
Copy link
Member

nikic commented Oct 7, 2019

@TysonAndre session issue should hopefully be fixed by 7d19668.

@TysonAndre TysonAndre deleted the image-opcache-info branch November 16, 2019 00:26
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

Successfully merging this pull request may close these issues.

2 participants