Skip to content

NewLanguageConstructs: improve T_ELLIPSIS error message text#803

Merged
wimg merged 1 commit intomasterfrom
feature/newlanguageconstructs-improve-error-message
Jun 16, 2019
Merged

NewLanguageConstructs: improve T_ELLIPSIS error message text#803
wimg merged 1 commit intomasterfrom
feature/newlanguageconstructs-improve-error-message

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented May 31, 2019

The T_ELLIPSIS or spread operator can be used for both variadic functions as well as for argument unpacking. Support for both was added in PHP 5.6.

In PHP 7.4, a third use case is added: unpacking arrays at array declaration.

This minor change is intended to make the language of the error message clearer so it will be applicable to all use-cases.

Related #808

The `T_ELLIPSIS` or spread operator can be used for both variadic functions as well as for argument unpacking. Support for both was added in PHP 5.6.

In PHP 7.4, a third use case is added: unpacking arrays at array declaration.

This minor change is intended to make the language of the error message clearer so it will be applicable to all use-cases.
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels May 31, 2019
@jrfnl jrfnl added this to the 9.x Next milestone May 31, 2019
@wimg wimg merged commit 58a222c into master Jun 16, 2019
@delete-merged-branch delete-merged-branch bot deleted the feature/newlanguageconstructs-improve-error-message branch June 16, 2019 13:25
@jrfnl jrfnl removed the PR: quick merge PR only contains relatively simple changes label Sep 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants