-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
Fixed some arg infos to match documentation #4139
Conversation
Hi, this should target |
|
Apart from the test failure those renames look fine. I think we should also update the |
5cffd47
to
0fe767d
Compare
@nikic I hope it's ok now. |
ZEND_ARG_INFO(0, a) | ||
ZEND_ARG_INFO(0, b) | ||
ZEND_ARG_INFO(0, value1) | ||
ZEND_ARG_INFO(0, value2) |
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.
Should update
Line 857 in 0fe767d
/* {{{ proto bool SplMinHeap::compare(mixed $a, mixed $b) |
I hope it's finally ok :) I'm sorry it's my first PR to PHP source and I'm only a PHP/Javascript developer :) |
Sorry for the delay, now merged as d6c0c5e. |
No description provided.