Skip to content

[NETBEANS-946] PHP 7.3 support (Flexible Heredoc and Nowdoc Syntaxes)#889

Merged
tmysik merged 2 commits intoapache:masterfrom
junichi11:php7.3-support-flexible-heredoc-nowdoc-syntaxes
Sep 20, 2018
Merged

[NETBEANS-946] PHP 7.3 support (Flexible Heredoc and Nowdoc Syntaxes)#889
tmysik merged 2 commits intoapache:masterfrom
junichi11:php7.3-support-flexible-heredoc-nowdoc-syntaxes

Conversation

@junichi11
Copy link
Member

@junichi11 junichi11 commented Sep 19, 2018

Implement Flexible Heredoc and Nowdoc Syntaxes.

  • Fix ASTPHPScanner and PHP5ColoringLexer to provide support for new syntaxes
  • Remove some existing unit tests because we won't provide support for the old syntax any longer
  • Show errors in PHP73UnhandleError when PHP version less than 7.3 is chosen
  • Add unit tests for the lexers, the parser, and the formatter

nb10-php73-flexible-heredoc-and-nowdoc-syntaxes

NOTE: Allow mixing spaces and tabs as indentation because the lexer would be complex if we handle it as errors there.

Move unit test data for "Function Call Trailing Commas" and "List Reference Assignment" to the php73 directory.

- Fix ASTPHPScanner and PHP5ColoringLexer to provide support for new syntaxes
- Remove some existing unit tests because we won't provide support for the old syntax any longer
- Show errors in PHP73UnhandleError when PHP version less than 7.3 is chosen
- Add unit tests for the lexers, the parser, and the formatter
- Function Call Trailing Commas
- List Reference Assignment
@junichi11 junichi11 added the PHP [ci] enable extra PHP tests (php/php.editor) label Sep 19, 2018
@junichi11 junichi11 requested a review from tmysik September 19, 2018 23:32
@geertjanw
Copy link
Member

Extremely cool.

Copy link
Member

@tmysik tmysik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect PR!

@tmysik tmysik merged commit 983e2eb into apache:master Sep 20, 2018
@junichi11 junichi11 deleted the php7.3-support-flexible-heredoc-nowdoc-syntaxes branch September 20, 2018 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PHP [ci] enable extra PHP tests (php/php.editor)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants