Skip to content

/ php-src

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

Jump to bottom

Fix more basic function stubs #6015

Closed
wants to merge 1 commit into from
Closed

Fix more basic function stubs #6015

wants to merge 1 commit into from

Conversation

@TysonAndre
Copy link
Contributor

@TysonAndre TysonAndre commented Aug 18, 2020 •
edited

User-defined functions can't have multiple parameters with the same name.
Don't do that for var_dump/debug_zval_dump.

Consistently use array $array to match docs
@TysonAndre
Fix more basic function stubs
Loading status checks…
87df1e0 
User-defined functions can't have multiple parameters with the same name.
Don't do that for var_dump/debug_zval_dump.

Consistently use array $array to match docs

Fix typo in UPGRADING
@TysonAndre TysonAndre force-pushed the TysonAndre:more-names branch from b0a9ec0 to 87df1e0 Aug 18, 2020
@kocsismate
kocsismate approved these changes Aug 19, 2020
View changes
@php-pulls php-pulls closed this in 9883fec Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kocsismate kocsismate

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@TysonAndre @kocsismate