Release PHP-Parser 4.18.0

Add ParserFactory::createForNewestSupportedVersion() and
ParserFactory::createForHostVersion() for forward-compatibility
with PHP-Parser 5. These methods do not accept an externally
constructed lexer and always enable all attributes.

.gitattributes

@@ -1,12 +1,11 @@
 /.github         export-ignore
 /doc             export-ignore
+/grammar         export-ignore
 /test            export-ignore
 /test_old        export-ignore
-/tools           export-ignore
 .editorconfig    export-ignore
 .gitattributes   export-ignore
 .gitignore       export-ignore
 CHANGELOG.md     export-ignore
-CONTRIBUTING.md  export-ignore
 phpunit.xml.dist export-ignore
 UPGRADE-*.md     export-ignore

.github/workflows/main.yml

@@ -5,17 +5,17 @@ on:
   pull_request:
 
 jobs:
-  tests_coverage:
+  tests_70:
     runs-on: "ubuntu-latest"
-    name: "PHP 7.1 Unit Tests (with coverage)"
+    name: "PHP 7.0 Unit Tests"
     steps:
       - name: "Checkout"
-        uses: "actions/checkout@v2"
+        uses: "actions/checkout@v3"
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
         with:
           coverage: "xdebug"
-          php-version: "7.1"
+          php-version: "7.0"
           tools: composer:v2
       - name: "Install dependencies"
         run: |
@@ -34,15 +34,17 @@ jobs:
     strategy:
       matrix:
         php-version:
+          - "7.1"
           - "7.2"
           - "7.3"
           - "7.4"
           - "8.0"
           - "8.1"
           - "8.2"
+          - "8.3"
     steps:
       - name: "Checkout"
-        uses: "actions/checkout@v2"
+        uses: "actions/checkout@v3"
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
         with:
@@ -58,7 +60,7 @@ jobs:
     name: "PHP 7.3 Code on PHP 8.0 Integration Tests"
     steps:
       - name: "Checkout"
-        uses: "actions/checkout@v2"
+        uses: "actions/checkout@v3"
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
         with:
@@ -71,38 +73,17 @@ jobs:
         run: "test_old/run-php-src.sh 7.3.21"
   test_old_80_70:
     runs-on: "ubuntu-latest"
-    name: "PHP 8.2 Code on PHP 7.1 Integration Tests"
+    name: "PHP 8.1 Code on PHP 7.0 Integration Tests"
     steps:
       - name: "Checkout"
-        uses: "actions/checkout@v2"
+        uses: "actions/checkout@v3"
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
         with:
           coverage: "none"
-          php-version: "7.1"
+          php-version: "7.0"
           tools: composer:v2
       - name: "Install PHP 8 dependencies"
         run: "composer update --no-progress --prefer-dist"
       - name: "Tests"
-        run: "test_old/run-php-src.sh 8.2.3"
-  phpstan:
-    runs-on: "ubuntu-latest"
-    name: "PHP ${{ matrix.php-version }} PHPStan"
-    strategy:
-      matrix:
-        php-version:
-          - "8.2"
-    steps:
-      - name: "Checkout"
-        uses: "actions/checkout@v2"
-      - name: "Install PHP"
-        uses: "shivammathur/setup-php@v2"
-        with:
-          coverage: "none"
-          php-version: "${{ matrix.php-version }}"
-          tools: composer:v2
-      - name: "Install dependencies"
-        run: | 
-          cd tools && composer install
-      - name: "PHPStan"
-        run: "php tools/vendor/bin/phpstan"
+        run: "test_old/run-php-src.sh 8.1.6"

.gitignore

@@ -1,7 +1,5 @@
-.idea/
 vendor/
 composer.lock
 grammar/kmyacc.exe
 grammar/y.output
 .phpunit.result.cache
-.php-cs-fixer.cache

.php-cs-fixer.dist.php

@@ -1,26 +0,0 @@
-<?php
-
-$finder = PhpCsFixer\Finder::create()
-    ->exclude('PhpParser/Parser')
-    ->in(__DIR__ . '/lib')
-    ->in(__DIR__ . '/test')
-    ->in(__DIR__ . '/grammar')
-;
-
-$config = new PhpCsFixer\Config();
-return $config->setRiskyAllowed(true)
-    ->setRules([
-        '@PSR12' => true,
-        // We use PSR12 with consistent brace placement.
-        'curly_braces_position' => [
-            'functions_opening_brace' => 'same_line',
-            'classes_opening_brace' => 'same_line',
-        ],
-        // declare(strict_types=1) on the same line as <?php.
-        'blank_line_after_opening_tag' => false,
-        'declare_strict_types' => true,
-        // Keep argument formatting for now.
-        'method_argument_space' => ['on_multiline' => 'ignore'],
-    ])
-    ->setFinder($finder)
-;

CHANGELOG.md

@@ -1,72 +1,85 @@
-Version 5.0.0-alpha2 (2023-03-05)
----------------------------------
-
-See UPGRADE-5.0 for detailed migration instructions.
+Version 4.18.0 (2023-12-10)
+---------------------------
 
 ### Added
 
-* [PHP 8.3] Added support for dynamic class constant fetch.
-* Added many additional type annotations. PhpStan is now used.
-* Added a fuzzing target for PHP-Fuzzer, which was how a lot of pretty printer bugs were found.
-* Added `isPromoted()`, `isPublic()`, `isProtected()`, `isPrivate()` and `isReadonly()` methods
-  on `Param`.
-* Added support for class constants in trait builder.
-* Added `PrettyPrinter` interface.
-* Added support for formatting preservation when toggling static modifiers.
-* The `php-parse` binary now accepts `-` as the file name, in which case it will read from stdin.
+* Added methods `ParserFactory::createForNewestSupportedVersion()` and
+  `ParserFactory::createForHostVersion()` for forward-compatibility with PHP-Parser 5.0.
 
 ### Fixed
 
-* The pretty printer now uses a more accurate treatment of unary operator precedence, and will only
-  wrap them in parentheses if required. This allowed fixing a number of other precedence related
-  bugs.
-* The pretty printer now respects the precedence of `clone`, `throw` and arrow functions.
-* The pretty printer no longer unconditionally wraps `yield` in parentheses, unless the target
-  version is set to older than PHP 7.0.
-* Fixed formatting preservation for alternative elseif/else syntax.
-* Fixed checks for when it is safe to print strings as heredoc/nowdoc to accommodate flexible
-  doc string semantics.
-* The pretty printer now prints parentheses around new/instanceof operands in all required
-  situations.
-* Similar, differences in allowed expressions on the LHS of `->` and `::` are now taken into account.
-* Fixed various cases where `\r` at the end of a doc string could be incorrectly merged into a CRLF
-  sequence with a following `\n`.
-* `__halt_compiler` is no longer recognized as a semi-reserved keyword, in line with PHP behavior.
-* `<?=` is no longer recognized as a semi-reserved keyword.
-* Fix handling of very large overflowing `\u` escape sequences.
-
-### Removed
-
-* Removed deprecated `Error` constructor taking a line number instead of an attributes array.
-
-Version 5.0.0-alpha1 (2022-09-04)
----------------------------------
-
-See UPGRADE-5.0 for detailed migration instructions.
+* Fixed missing name resolution of class constant types.
+* Fixed class members being dropped if an error is encountered while parsing a later class member
+  (when error recovery is enabeld).
 
 ### Changed
 
-* PHP 7.1 is now required to run PHP-Parser.
-* Formatting of the standard pretty printer has been adjusted to match PSR-12 more closely.
-* The internal token representation now uses a `PhpParser\Token` class, which is compatible with
-  PHP 8 token representation (`PhpToken`).
-* Destructuring is now always represented using `Expr\List_` nodes, even if it uses `[]` syntax.
-* Renamed a number of node classes, and moved things that were not real expressions/statements
-  outside the `Expr`/`Stmt` hierarchy. Compatibility shims for the old names have been retained.
+* The `grammar/` directory has been excluded from exported git archives.
+
+Version 4.17.1 (2023-08-13)
+---------------------------
+
+### Fixed
+
+* Fixed phpdoc mismatches for `ClassConst::$type` introduced in previous release.
+
+Version 4.17.0 (2023-08-13)
+---------------------------
 
 ### Added
 
-* Added `PhpVersion` class, which is accepted in a number of places (e.g. ParserFactory, Parser,
-  Lexer, PrettyPrinter) and gives more precise control over the PHP version being targeted.
-* Added PHP 8 parser though it only differs from the PHP 7 parser in concatenation precedence.
-* Added `Parser::getLexer()` method.
-* Added a `Modifiers` class, as a replacement for `Stmt\Class_::MODIFIER_*`.
-* Added support for returning an array or `REMOVE_NODE` from `NodeVisitor::enterNode()`.
+* [PHP 8.3] Added support for typed class constants.
+* [PHP 8.3] Added supprot for dynamic class constant fetch.
+* [PHP 8.3] Added support for readonly anonymous classes.
 
-### Removed
+### Fixed
 
-* The PHP 5 parser has been removed. The PHP 7 parser has been adjusted to deal with PHP 5 code
-  more gracefully.
+* Fixed missing required parentheses when pretty printing new with an expression class name.
+* Fixed missing required parentheses when pretty printing `(CONST)::$x` and similar.
+
+Version 4.16.0 (2023-06-25)
+---------------------------
+
+### Added
+
+* Added `Name::getParts()` method for forward-compatibility with PHP-Parser 5.
+
+### Deprecated
+
+* Deprecated direct access to `Name::$parts`, which will be removed in PHP-Parser 5.
+
+Version 4.15.5 (2023-05-19)
+---------------------------
+
+### Added
+
+* Added `makePrivate()`, `makeProtected()`, `makePublic()` and `makeReadonly()` methods to
+  `Builder\Param` to allow the creation of promoted parameters.
+
+Version 4.15.4 (2023-03-05)
+---------------------------
+
+### Fixed
+
+* Fixed formatting-preservation for alternative if syntax with trailing comments.
+
+Version 4.15.3 (2023-01-16)
+---------------------------
+
+### Fixed
+
+* Support readonly property with PHP 8.2 DNF type.
+* Fixed PHP attribute group and PHP-Parser attribute mixup in EnumCase builder.
+
+Version 4.15.2 (2022-11-12)
+---------------------------
+
+### Fixed
+
+* Fixed parsing of large hex float literals that contain an "e" character.
+* Fixed tests to pass on 32-bit.
+* Fixed generation of invalid code when using formatting-preserving pretty printer with code that
+  uses inline HTML.
 
 Version 4.15.1 (2022-09-04)
 ---------------------------

CONTRIBUTING.md

@@ -1,4 +0,0 @@
-## Coding Style
-
-This project uses PSR-12 with consistent brace placement. This means that the opening brace is
-always on the same line, even for class and method declarations.

README.md

@@ -3,11 +3,9 @@ PHP Parser
 
 [![Coverage Status](https://coveralls.io/repos/github/nikic/PHP-Parser/badge.svg?branch=master)](https://coveralls.io/github/nikic/PHP-Parser?branch=master)
 
-This is a PHP parser written in PHP. Its purpose is to simplify static code analysis and
+This is a PHP 5.2 to PHP 8.2 parser written in PHP. Its purpose is to simplify static code analysis and
 manipulation.
 
-[Documentation for version 5.x][doc_master] (in development; for running on PHP >= 7.1; for parsing PHP 7.0 to PHP 8.2, with limited support for parsing PHP 5.x).
-
 [**Documentation for version 4.x**][doc_4_x] (stable; for running on PHP >= 7.0; for parsing PHP 5.2 to PHP 8.2).
 
 [Documentation for version 3.x][doc_3_x] (unsupported; for running on PHP >= 5.5; for parsing PHP 5.2 to PHP 7.2).
@@ -17,12 +15,12 @@ Features
 
 The main features provided by this library are:
 
- * Parsing PHP 7, and PHP 8 code into an abstract syntax tree (AST).
+ * Parsing PHP 5, PHP 7, and PHP 8 code into an abstract syntax tree (AST).
    * Invalid code can be parsed into a partial AST.
    * The AST contains accurate location information.
  * Dumping the AST in human-readable form.
  * Converting an AST back to PHP code.
-   * Formatting can be preserved for partially changed ASTs.
+   * Experimental: Formatting can be preserved for partially changed ASTs.
  * Infrastructure to traverse and modify ASTs.
  * Resolution of namespaced names.
  * Evaluation of constant expressions.
@@ -53,7 +51,7 @@ function test($foo)
 }
 CODE;
 
-$parser = (new ParserFactory())->createForNewestSupportedVersion();
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 try {
     $ast = $parser->parse($code);
 } catch (Error $error) {
@@ -225,4 +223,3 @@ Component documentation:
 
  [doc_3_x]: https://github.com/nikic/PHP-Parser/tree/3.x/doc
  [doc_4_x]: https://github.com/nikic/PHP-Parser/tree/4.x/doc
- [doc_master]: https://github.com/nikic/PHP-Parser/tree/master/doc

UPGRADE-5.0.md

@@ -1,223 +0,0 @@
-Upgrading from PHP-Parser 4.x to 5.0
-====================================
-
-### PHP version requirements
-
-PHP-Parser now requires PHP 7.1 or newer to run. It is however still possible to *parse* code for older versions, while running on a newer version.
-
-### PHP 5 parsing support
-
-The dedicated parser for PHP 5 has been removed. The PHP 7 parser now accepts a `PhpVersion` argument, which can be used to improve compatibility with older PHP versions.
-
-In particular, if an older `PhpVersion` is specified, then:
-
- * For versions before PHP 7.0, `$foo =& new Bar()` assignments are allowed without error.
- * For versions before PHP 7.0, invalid octal literals `089` are allowed without error.
- * Type hints are interpreted as a class `Name` or as a built-in `Identifier` depending on PHP
-   version, for example `int` is treated as a class name on PHP 5.6 and as a built-in on PHP 7.0.
-
-However, some aspects of PHP 5 parsing are no longer supported:
-
- * Some variables like `$$foo[0]` are valid in both PHP 5 and PHP 7, but have different interpretation. In that case, the PHP 7 AST will always be constructed (`($$foo)[0]` rather than `${$foo[0]}`).
- * Declarations of the form `global $$var[0]` are not supported in PHP 7 and will cause a parse error. In error recovery mode, it is possible to continue parsing after such declarations.
- * The PHP 7 parser will accept many constructs that are not valid in PHP 5. However, this was also true of the dedicated PHP 5 parser.
-
-The following symbols are affected by this removal:
-
- * The `PhpParser\Parser\Php5` class has been removed.
- * The `PhpParser\Parser\Multiple` class has been removed. While not strictly related to PHP 5 support, this functionality is no longer useful without it.
- * The `PhpParser\ParserFactory::ONLY_PHP5` and `PREFER_PHP5` options have been removed.
- * The `PhpParser\ParserFactory::PREFER_PHP7` option is now equivalent to `ONLY_PHP7`.
-
-### Changes to the parser factory
-
-The `ParserFactory::create()` method is deprecated in favor of three new methods that provide more fine-grained control over the PHP version being targeted:
-
- * `createForNewestSupportedVersion()`: Use this if you don't know the PHP version of the code you're parsing. It's better to assume a too new version than a too old one.
- * `createForHostVersion()`: Use this if you're parsing code for the PHP version you're running on.
- * `createForVersion()`: Use this if you know the PHP version of the code you want to parse.
-
-In all cases, the PHP version is a fairly weak hint that is only used on a best-effort basis. The parser will usually accept code for newer versions if it does not have any backwards-compatibility implications.
-
-For example, if you specify version `"8.0"`, then `class ReadOnly {}` is treated as a valid class declaration, while using `public readonly int $prop` will lead to a parse error. However, `final public const X = Y;` will be accepted in both cases.
-
-```php
-use PhpParser\ParserFactory;
-use PhpParser\PhpVersion;
-
-$factory = new ParserFactory;
-
-# Before
-$parser = $factory->create(ParserFactory::PREFER_PHP7);
-
-# After (this is roughly equivalent to PREFER_PHP7 behavior)
-$parser = $factory->createForNewestSupportedVersion();
-# Or
-$parser = $factory->createForHostVersion();
-
-# Before
-$parser = $factory->create(ParserFactory::ONLY_PHP5);
-# After (supported on a best-effort basis)
-$parser = $factory->createForVersion(PhpVersion::fromString("5.6"));
-```
-
-### Changes to the array destructuring representation
-
-Previously, the `list($x) = $y` destructuring syntax was represented using a `Node\Expr\List_`
-node, while `[$x] = $y` used a `Node\Expr\Array_` node, the same used for the creation (rather than
-destructuring) of arrays.
-
-Now, destructuring is always represented using `Node\Expr\List_`. The `kind` attribute with value
-`Node\Expr\List_::KIND_LIST` or `Node\Expr\List_::KIND_ARRAY` specifies which syntax was actually
-used.
-
-### Renamed nodes
-
-A number of AST nodes have been renamed or moved in the AST hierarchy:
-
- * `Node\Scalar\LNumber` is now `Node\Scalar\Int_`.
- * `Node\Scalar\DNumber` is now `Node\Scalar\Float_`.
- * `Node\Scalar\Encapsed` is now `Node\Scalar\InterpolatedString`.
- * `Node\Scalar\EncapsedStringPart` is now `Node\InterpolatedStringPart` and no longer extends
-   `Node\Scalar` or `Node\Expr`.
- * `Node\Expr\ArrayItem` is now `Node\ArrayItem` and no longer extends `Node\Expr`.
- * `Node\Expr\ClosureUse` is now `Node\ClosureUse` and no longer extends `Node\Expr`.
- * `Node\Stmt\DeclareDeclare` is now `Node\DeclareItem` and no longer extends `Node\Stmt`.
- * `Node\Stmt\PropertyProperty` is now `Node\PropertyItem` and no longer extends `Node\Stmt`.
- * `Node\Stmt\StaticVar` is now `Node\StaticVar` and no longer extends `Node\Stmt`.
- * `Node\Stmt\UseUse` is now `Node\UseItem` and no longer extends `Node\Stmt`.
-
-The old class names have been retained as aliases for backwards compatibility. However, the `Node::getType()` method will now always return the new name (e.g. `ClosureUse` instead of `Expr_ClosureUse`).
-
-### Modifiers
-
-Modifier flags (as used by the `$flags` subnode of `Class_`, `ClassMethod`, `Property`, etc.) are now available as class constants on a separate `PhpParser\Modifiers` class, instead of being part of `PhpParser\Node\Stmt\Class_`, to make it clearer that these are used by many different nodes. The old constants are deprecated, but are still available.
-
-```
-PhpParser\Node\Stmt\Class_::MODIFIER_PUBLIC    -> PhpParser\Modifiers::PUBLIC
-PhpParser\Node\Stmt\Class_::MODIFIER_PROTECTED -> PhpParser\Modifiers::PROTECTED
-PhpParser\Node\Stmt\Class_::MODIFIER_PRIVATE   -> PhpParser\Modifiers::PRIVATE
-PhpParser\Node\Stmt\Class_::MODIFIER_STATIC    -> PhpParser\Modifiers::STATIC
-PhpParser\Node\Stmt\Class_::MODIFIER_ABSTRACT  -> PhpParser\Modifiers::ABSTRACT
-PhpParser\Node\Stmt\Class_::MODIFIER_FINAL     -> PhpParser\Modifiers::FINAL
-PhpParser\Node\Stmt\Class_::MODIFIER_READONLY  -> PhpParser\Modifiers::READONLY
-PhpParser\Node\Stmt\Class_::VISIBILITY_MODIFIER_MASK -> PhpParser\Modifiers::VISIBILITY_MASK
-```
-
-### Changes to the pretty printer
-
-A number of changes to the standard pretty printer have been made, to make it match contemporary coding style conventions (and in particular PSR-12). Options to restore the previous behavior are not provided, but it is possible to override the formatting methods (such as `pStmt_ClassMethod`) with your preferred formatting.
-
-Return types are now formatted without a space before the `:`:
-
-```php
-# Before
-function test() : Type
-{
-}
-
-# After
-function test(): Type
-{
-}
-```
-
-`abstract` and `final` are now printed before visibility modifiers:
-
-```php
-# Before
-public abstract function test();
-
-# After
-abstract public function test();
-```
-
-A space is now printed between `use` and the following `(` for closures:
-
-```php
-# Before
-function () use($var) {
-};
-
-# After
-function () use ($var) {
-};
-```
-
-Backslashes in single-quoted strings are now only printed if they are necessary:
-
-```php
-# Before
-'Foo\\Bar';
-'\\\\';
-
-# After
-'Foo\Bar';
-'\\\\';
-```
-
-The pretty printer now accepts a `phpVersion` option, which accepts a `PhpVersion` object and defaults to PHP 7.0. The pretty printer will make formatting choices to make the code valid for that version. It currently controls the following behavior:
-
-* For PHP >= 7.0 (default), short array syntax `[]` will be used by default. This does not affect nodes that specify an explicit array syntax using the `kind` attribute.
-* For PHP >= 7.0 (default), parentheses around `yield` expressions will only be printed when necessary. Previously, parentheses were always printed, even if `yield` was used as a statement.
-* For PHP >= 7.1, the short array syntax `[]` will be used for destructuring by default (instead of
-  `list()`). This does not affect nodes that specify and explicit syntax using the `kind` attribute.
-* For PHP >= 7.3, a newline is no longer forced after heredoc/nowdoc strings, as the requirement for this has been removed with the introduction of flexible heredoc/nowdoc strings.
-
-### Changes to precedence handling in the pretty printer
-
-The pretty printer now more accurately models operator precedence. Especially for unary operators, less unnecessary parentheses will be printed. Conversely, many bugs where semantically meaningful parentheses were omitted have been fixed.
-
-To support these changes, precedence is now handled differently in the pretty printer. The internal `p()` method, which is used to recursively print nodes, now has the following signature:
-```php
-protected function p(
-    Node $node, int $precedence = self::MAX_PRECEDENCE, int $lhsPrecedence = self::MAX_PRECEDENCE,
-    bool $parentFormatPreserved = false
-): string;
-```
-
-The `$precedence` is the precedence of the direct parent operator (if any), while `$lhsPrecedence` is that precedence of the nearest binary operator on whose left-hand-side the node occurs. For unary operators, only the `$lhsPrecedence` is relevant.
-
-Recursive calls in pretty-printer methods should generally continue calling `p()` without additional parameters. However, pretty-printer methods for operators that participate in precedence resolution need to be adjusted. For example, typical implementations for operators looks as follows now:
-
-```php
-protected function pExpr_BinaryOp_Plus(
-    BinaryOp\Plus $node, int $precedence, int $lhsPrecedence
-): string {
-    return $this->pInfixOp(
-        BinaryOp\Plus::class, $node->left, ' + ', $node->right, $precedence, $lhsPrecedence);
-}
-
-protected function pExpr_UnaryPlus(
-    Expr\UnaryPlus $node, int $precedence, int $lhsPrecedence
-): string {
-    return $this->pPrefixOp(Expr\UnaryPlus::class, '+', $node->expr, $precedence, $lhsPrecedence);
-}
-```
-
-The new `$precedence` and `$lhsPrecedence` arguments need to be passed down to the `pInfixOp()`, `pPrefixOp()` and `pPostfixOp()` methods.
-
-### Changes to token representation
-
-Tokens are now internally represented using the `PhpParser\Token` class, which exposes the same base interface as
-the `PhpToken` class introduced in PHP 8.0. On PHP 8.0 or newer, `PhpParser\Token` extends from `PhpToken`, otherwise
-it extends from a polyfill implementation. The most important parts of the interface may be summarized as follows:
-
-```php
-class Token {
-    public int $id;
-    public string $text;
-    public int $line;
-    public int $pos;
-
-    public function is(int|string|array $kind): bool;
-}
-```
-
-The `Lexer::getTokens()` method will now return an array of `Token`s, rather than an array of arrays and strings.
-Additionally, the token array is now terminated by a sentinel token with ID 0.
-
-### Other removed functionality
-
- * The deprecated `Builder\Param::setTypeHint()` method has been removed in favor of `Builder\Param::setType()`.
- * The deprecated `Error` constructor taking a start line has been removed. Pass `['startLine' => $startLine]` attributes instead.

bin/php-parse

@@ -26,10 +26,13 @@ if (empty($files)) {
     showHelp("Must specify at least one file.");
 }
 
-$lexerOptions = ['usedAttributes' => [
+$lexer = new PhpParser\Lexer\Emulative(['usedAttributes' => [
     'startLine', 'endLine', 'startFilePos', 'endFilePos', 'comments'
-]];
-$parser = (new PhpParser\ParserFactory())->createForVersion($attributes['version'], $lexerOptions);
+]]);
+$parser = (new PhpParser\ParserFactory)->create(
+    PhpParser\ParserFactory::PREFER_PHP7,
+    $lexer
+);
 $dumper = new PhpParser\NodeDumper([
     'dumpComments' => true,
     'dumpPositions' => $attributes['with-positions'],
@@ -40,10 +43,7 @@ $traverser = new PhpParser\NodeTraverser();
 $traverser->addVisitor(new PhpParser\NodeVisitor\NameResolver);
 
 foreach ($files as $file) {
-    if ($file === '-') {
-        $code = file_get_contents('php://stdin');
-        fwrite(STDERR, "====> Stdin:\n");
-    } else if (strpos($file, '<?php') === 0) {
+    if (strpos($file, '<?php') === 0) {
         $code = $file;
         fwrite(STDERR, "====> Code $code\n");
     } else {
@@ -108,7 +108,7 @@ function showHelp($error = '') {
     if ($error) {
         fwrite(STDERR, $error . "\n\n");
     }
-    fwrite($error ? STDERR : STDOUT, <<<'OUTPUT'
+    fwrite($error ? STDERR : STDOUT, <<<OUTPUT
 Usage: php-parse [operations] file1.php [file2.php ...]
    or: php-parse [operations] "<?php code"
 Turn PHP source code into an abstract syntax tree.
@@ -123,7 +123,6 @@ Operations is a list of the following options (--dump by default):
     -c, --with-column-info  Show column-numbers for errors (if available)
     -P, --with-positions    Show positions in node dumps
     -r, --with-recovery     Use parsing with error recovery
-        --version=VERSION   Target specific PHP version (default: newest)
     -h, --help              Display this page
 
 Example:
@@ -144,7 +143,6 @@ function parseArgs($args) {
         'with-column-info' => false,
         'with-positions' => false,
         'with-recovery' => false,
-        'version' => PhpParser\PhpVersion::getNewestSupported(),
     ];
 
     array_shift($args);
@@ -195,9 +193,7 @@ function parseArgs($args) {
                 $parseOptions = false;
                 break;
             default:
-                if (preg_match('/^--version=(.*)$/', $arg, $matches)) {
-                    $attributes['version'] = PhpParser\PhpVersion::fromString($matches[1]);
-                } elseif ($arg[0] === '-' && \strlen($arg[0]) > 1) {
+                if ($arg[0] === '-') {
                     showHelp("Invalid operation $arg.");
                 } else {
                     $files[] = $arg;

composer.json

@@ -13,18 +13,16 @@
         }
     ],
     "require": {
-        "php": ">=7.1",
-        "ext-tokenizer": "*",
-        "ext-json": "*",
-        "ext-ctype": "*"
+        "php": ">=7.0",
+        "ext-tokenizer": "*"
     },
     "require-dev": {
-        "phpunit/phpunit": "^7.0 || ^8.0 || ^9.0",
+        "phpunit/phpunit": "^6.5 || ^7.0 || ^8.0 || ^9.0",
         "ircmaxell/php-yacc": "^0.0.7"
     },
     "extra": {
         "branch-alias": {
-            "dev-master": "5.0-dev"
+            "dev-master": "4.9-dev"
         }
     },
     "autoload": {

doc/0_Introduction.markdown

@@ -1,7 +1,7 @@
 Introduction
 ============
 
-This project is a PHP parser **written in PHP itself**.
+This project is a PHP 5.2 to PHP 8.0 parser **written in PHP itself**.
 
 What is this for?
 -----------------
@@ -26,29 +26,16 @@ programmatic PHP code analysis are incidentally PHP developers, not C developers
 What can it parse?
 ------------------
 
-The parser supports parsing PHP 7 and PHP 8 code, with the following exceptions:
+The parser supports parsing PHP 5.2-8.0, with the following exceptions:
 
  * Namespaced names containing whitespace (e.g. `Foo \ Bar` instead of `Foo\Bar`) are not supported.
-   These are illegal in PHP 8, but are legal in earlier versions. However, PHP-Parser does not
+   These are illegal in PHP 8, but are legal in earlier version. However, PHP-Parser does not
    support them for any version.
 
-PHP-Parser 4.x had full support for parsing PHP 5. PHP-Parser 5.x has only limited support, with the
-following caveats:
-
- * Some variable expressions like `$$foo[0]` are valid in both PHP 5 and PHP 7, but have different 
-   interpretation. In such cases, the PHP 7 AST will always be constructed (using `($$foo)[0]`
-   rather than `${$foo[0]}`).
- * Declarations of the form `global $$var[0]` are not supported in PHP 7 and will cause a parse 
-   error. In error recovery mode, it is possible to continue parsing after such declarations.
-
 As the parser is based on the tokens returned by `token_get_all` (which is only able to lex the PHP
 version it runs on), additionally a wrapper for emulating tokens from newer versions is provided.
-This allows to parse PHP 8.0 source code running on PHP 7.1, for example. This emulation is not
-perfect, but works well in practice.
-
-Finally, it should be noted that the parser aims to accept all valid code, not reject all invalid
-code. It will generally accept code that is only valid in newer versions (even when targeting an
-older one), and accept code that is syntactically correct, but would result in a compiler error.
+This allows to parse PHP 7.4 source code running on PHP 7.0, for example. This emulation is somewhat
+hacky and not perfect, but it should work well on any sane code.
 
 What output does it produce?
 ----------------------------
@@ -76,7 +63,7 @@ This matches the structure of the code: An echo statement, which takes two strin
 with the values `Hi` and `World`.
 
 You can also see that the AST does not contain any whitespace information (but most comments are saved).
-However, it does retain accurate position information, which can be used to inspect precise formatting.
+So using it for formatting analysis is not possible.
 
 What else can it do?
 --------------------
@@ -87,7 +74,7 @@ Apart from the parser itself this package also bundles support for some other, r
    that "pretty printing" does not imply that the output is especially pretty. It's just how it's
    called ;)
  * Support for serializing and unserializing the node tree to JSON
- * Support for dumping the node tree in a human-readable form (see the section above for an
+ * Support for dumping the node tree in a human readable form (see the section above for an
    example of how the output looks like)
  * Infrastructure for traversing and changing the AST (node traverser and node visitors)
  * A node visitor for resolving namespaced names

doc/2_Usage_of_basic_components.markdown

@@ -12,7 +12,7 @@ To bootstrap the library, include the autoloader generated by composer:
 require 'path/to/vendor/autoload.php';
 ```
 
-Additionally, you may want to set the `xdebug.max_nesting_level` ini option to a higher value:
+Additionally you may want to set the `xdebug.max_nesting_level` ini option to a higher value:
 
 ```php
 ini_set('xdebug.max_nesting_level', 3000);
@@ -29,29 +29,25 @@ In order to parse code, you first have to create a parser instance:
 
 ```php
 use PhpParser\ParserFactory;
-use PhpParser\PhpVersion;
-
-// Parser for the version you are running on.
-$parser = (new ParserFactory())->createForHostVersion();
-
-// Parser for the newest PHP version supported by the PHP-Parser library.
-$parser = (new ParserFactory())->createForNewestSupportedVersion();
-
-// Parser for a specific PHP version.
-$parser = (new ParserFactory())->createForVersion(PhpVersion::fromString('8.1'));
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 ```
 
-Which version you should target depends on your use case. In many cases you will want to use the
-host version, as people typically analyze code for the version they are running on. However, when
-analyzing arbitrary code you are usually best off using the newest supported version, which tends
-to accept the widest range of code (unless there are breaking changes in PHP).
+The factory accepts a kind argument, that determines how different PHP versions are treated:
 
-The `createXYZ()` methods optionally accept an array of lexer options. Some use cases that require
-customized lexer options are discussed in the [lexer documentation](component/Lexer.markdown).
+Kind | Behavior
+-----|---------
+`ParserFactory::PREFER_PHP7` | Try to parse code as PHP 7. If this fails, try to parse it as PHP 5.
+`ParserFactory::PREFER_PHP5` | Try to parse code as PHP 5. If this fails, try to parse it as PHP 7.
+`ParserFactory::ONLY_PHP7` | Parse code as PHP 7.
+`ParserFactory::ONLY_PHP5` | Parse code as PHP 5.
 
-Subsequently, you can pass PHP code (including the opening `<?php` tag) to the `parse()` method in
-order to create a syntax tree. If a syntax error is encountered, an `PhpParser\Error` exception will
-be thrown by default:
+Unless you have a strong reason to use something else, `PREFER_PHP7` is a reasonable default.
+
+The `create()` method optionally accepts a `Lexer` instance as the second argument. Some use cases
+that require customized lexers are discussed in the [lexer documentation](component/Lexer.markdown).
+
+Subsequently you can pass PHP code (including the opening `<?php` tag) to the `parse` method in order to
+create a syntax tree. If a syntax error is encountered, an `PhpParser\Error` exception will be thrown:
 
 ```php
 <?php
@@ -66,7 +62,7 @@ function printLine($msg) {
 printLine('Hello World!!!');
 CODE;
 
-$parser = (new ParserFactory())->createForHostVersion();
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 
 try {
     $stmts = $parser->parse($code);
@@ -81,7 +77,7 @@ A parser instance can be reused to parse multiple files.
 Node dumping
 ------------
 
-To dump the abstract syntax tree in human-readable form, a `NodeDumper` can be used:
+To dump the abstract syntax tree in human readable form, a `NodeDumper` can be used:
 
 ```php
 <?php
@@ -175,7 +171,7 @@ with them easier they are grouped into three categories:
 
  * `PhpParser\Node\Stmt`s are statement nodes, i.e. language constructs that do not return
    a value and can not occur in an expression. For example a class definition is a statement.
-   It doesn't return a value, and you can't write something like `func(class A {});`.
+   It doesn't return a value and you can't write something like `func(class A {});`.
  * `PhpParser\Node\Expr`s are expression nodes, i.e. language constructs that return a value
    and thus can occur in other expressions. Examples of expressions are `$var`
    (`PhpParser\Node\Expr\Variable`) and `func()` (`PhpParser\Node\Expr\FuncCall`).
@@ -205,14 +201,15 @@ can then be retrieved using `hasAttribute()`, `getAttribute()` and `getAttribute
 By default the lexer adds the `startLine`, `endLine` and `comments` attributes. `comments` is an array
 of `PhpParser\Comment[\Doc]` instances.
 
-The start line can also be accessed using `getStartLine()` (instead of `getAttribute('startLine')`).
+The start line can also be accessed using `getLine()`/`setLine()` (instead of `getAttribute('startLine')`).
 The last doc comment from the `comments` attribute can be obtained using `getDocComment()`.
 
 Pretty printer
 --------------
 
-The pretty printer component compiles the AST back to PHP code according to a specified scheme.
-Currently, there is only one scheme available, namely `PhpParser\PrettyPrinter\Standard`.
+The pretty printer component compiles the AST back to PHP code. As the parser does not retain formatting
+information the formatting is done using a specified scheme. Currently there is only one scheme available,
+namely `PhpParser\PrettyPrinter\Standard`.
 
 ```php
 use PhpParser\Error;
@@ -221,8 +218,8 @@ use PhpParser\PrettyPrinter;
 
 $code = "<?php echo 'Hi ', hi\\getTarget();";
 
-$parser = (new ParserFactory())->createForHostVersion();
-$prettyPrinter = new PrettyPrinter\Standard();
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
+$prettyPrinter = new PrettyPrinter\Standard;
 
 try {
     // parse
@@ -257,13 +254,10 @@ single expression using `prettyPrintExpr()`.
 The `prettyPrintFile()` method can be used to print an entire file. This will include the opening `<?php` tag
 and handle inline HTML as the first/last statement more gracefully.
 
-There is also a pretty-printing mode which retains formatting for parts of the AST that have not
-been changed, which requires additional setup.
-
 > Read more: [Pretty printing documentation](component/Pretty_printing.markdown)
 
-Node traversal
---------------
+Node traversation
+-----------------
 
 The above pretty printing example used the fact that the source code was known and thus it was easy to
 write code that accesses a certain part of a node tree and changes it. Normally this is not the case.
@@ -278,7 +272,7 @@ use PhpParser\NodeTraverser;
 use PhpParser\ParserFactory;
 use PhpParser\PrettyPrinter;
 
-$parser        = (new ParserFactory())->createForHostVersion();
+$parser        = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 $traverser     = new NodeTraverser;
 $prettyPrinter = new PrettyPrinter\Standard;
 
@@ -309,7 +303,8 @@ The corresponding node visitor might look like this:
 use PhpParser\Node;
 use PhpParser\NodeVisitorAbstract;
 
-class MyNodeVisitor extends NodeVisitorAbstract {
+class MyNodeVisitor extends NodeVisitorAbstract
+{
     public function leaveNode(Node $node) {
         if ($node instanceof Node\Scalar\String_) {
             $node->value = 'foo';
@@ -331,7 +326,7 @@ public function afterTraverse(array $nodes);
 ```
 
 The `beforeTraverse()` method is called once before the traversal begins and is passed the nodes the
-traverser was called with. This method can be used for resetting values before traversal or
+traverser was called with. This method can be used for resetting values before traversation or
 preparing the tree for traversal.
 
 The `afterTraverse()` method is similar to the `beforeTraverse()` method, with the only difference that
@@ -347,8 +342,8 @@ The `enterNode()` method can additionally return the value `NodeTraverser::DONT_
 which instructs the traverser to skip all children of the current node. To furthermore prevent subsequent
 visitors from visiting the current node, `NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN` can be used instead.
 
-Both methods can additionally return the value `NodeTraverser::REMOVE_NODE`, in which
-case the current node will be removed from the parent array. Furthermore, it is possible to return
+The `leaveNode()` method can additionally return the value `NodeTraverser::REMOVE_NODE`, in which
+case the current node will be removed from the parent array. Furthermore it is possible to return
 an array of nodes, which will be merged into the parent array at the offset of the current node.
 I.e. if in `array(A, B, C)` the node `B` should be replaced with `array(X, Y, Z)` the result will
 be `array(A, X, Y, Z, C)`.
@@ -377,9 +372,8 @@ unqualified function and constant names. These are resolved at runtime and thus
 know which function they are referring to. In most cases this is a non-issue as the global functions
 are meant.
 
-Additionally, the `NameResolver` adds a `namespacedName` subnode to class, function and constant
-declarations that contains the namespaced name instead of only the shortname that is available via
-`name`.
+Also the `NameResolver` adds a `namespacedName` subnode to class, function and constant declarations
+that contains the namespaced name instead of only the shortname that is available via `name`.
 
 > Read more: [Name resolution documentation](component/Name_resolution.markdown)
 
@@ -402,7 +396,7 @@ use PhpParser\NodeVisitor\NameResolver;
 $inDir  = '/some/path';
 $outDir = '/some/other/path';
 
-$parser        = (new ParserFactory())->createForNewestSupportedVersion();
+$parser        = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 $traverser     = new NodeTraverser;
 $prettyPrinter = new PrettyPrinter\Standard;
 

doc/component/AST_builders.markdown

@@ -101,7 +101,7 @@ abstract class SomeOtherClass extends SomeClass implements A\Few, \Interfaces
      *
      * @param SomeClass And takes a parameter
      */
-    public abstract function someMethod(SomeClass $someParam): bool;
+    public abstract function someMethod(SomeClass $someParam) : bool;
     protected function anotherMethod($someParam = 'test')
     {
         print $someParam;

doc/component/Constant_expression_evaluation.markdown

@@ -19,9 +19,9 @@ PHP-Parser supports evaluation of such constant expressions through the `ConstEx
 
 use PhpParser\{ConstExprEvaluator, ConstExprEvaluationException};
 
-$evaluator = new ConstExprEvaluator();
+$evalutator = new ConstExprEvaluator();
 try {
-    $value = $evaluator->evaluateSilently($someExpr);
+    $value = $evalutator->evaluateSilently($someExpr);
 } catch (ConstExprEvaluationException $e) {
     // Either the expression contains unsupported expression types,
     // or an error occurred during evaluation
@@ -45,7 +45,7 @@ use PhpParser\Node\{Expr, Scalar};
 $evaluator = new ConstExprEvaluator();
 
 // 10 / 0
-$expr = new Expr\BinaryOp\Div(new Scalar\Int_(10), new Scalar\Int_(0));
+$expr = new Expr\BinaryOp\Div(new Scalar\LNumber(10), new Scalar\LNumber(0));
 
 var_dump($evaluator->evaluateDirectly($expr)); // float(INF)
 // Warning: Division by zero
@@ -69,7 +69,6 @@ expressions, apart from the following:
  * `Scalar\MagicConst\*`
  * `Expr\ConstFetch` (only null/false/true are handled)
  * `Expr\ClassConstFetch`
- * `Expr\New_` (since PHP 8.1)
 
 Handling these expression types requires non-local information, such as which global constants are
 defined. By default, the evaluator will throw a `ConstExprEvaluationException` when it encounters
@@ -113,4 +112,4 @@ class Test {
     const A = self::B;
     const B = self::A;
 }
-```
+```

doc/component/Error_handling.markdown

@@ -14,10 +14,10 @@ In order to receive information about not only the line, but also the column spa
 position attributes in the lexer need to be enabled:
 
 ```php
-$lexerOptions = array(
+$lexer = new PhpParser\Lexer(array(
     'usedAttributes' => array('comments', 'startLine', 'endLine', 'startFilePos', 'endFilePos'),
-);
-$parser = (new PhpParser\ParserFactory())->createForHostVersion($lexerOptions);
+));
+$parser = (new PhpParser\ParserFactory)->create(PhpParser\ParserFactory::PREFER_PHP7, $lexer);
 
 try {
     $stmts = $parser->parse($code);
@@ -56,7 +56,7 @@ To instead collect all encountered errors into an array, while trying to continu
 an instance of `ErrorHandler\Collecting` can be passed to the `Parser::parse()` method. A usage example:
 
 ```php
-$parser = (new PhpParser\ParserFactory())->createForHostVersion();
+$parser = (new PhpParser\ParserFactory)->create(PhpParser\ParserFactory::ONLY_PHP7);
 $errorHandler = new PhpParser\ErrorHandler\Collecting;
 
 $stmts = $parser->parse($code, $errorHandler);
@@ -72,6 +72,4 @@ if (null !== $stmts) {
 }
 ```
 
-The partial AST may contain `Expr\Error` nodes that indicate that an error occurred while parsing an expression.
-
 The `NameResolver` visitor also accepts an `ErrorHandler` as a constructor argument.

doc/component/FAQ.markdown

@@ -19,7 +19,7 @@ $code = '...';
 $traverser = new NodeTraverser;
 $traverser->addVisitor(new ParentConnectingVisitor);
 
-$parser = (new ParserFactory())->createForHostVersion();
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 $ast    = $parser->parse($code);
 $ast    = $traverser->traverse($ast);
 ```
@@ -42,7 +42,7 @@ $code = '...';
 $traverser = new NodeTraverser;
 $traverser->addVisitor(new NodeConnectingVisitor);
 
-$parser = (new ParserFactory())->createForHostVersion();
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 $ast    = $parser->parse($code);
 $ast    = $traverser->traverse($ast);
 ```

doc/component/JSON_representation.markdown

@@ -18,7 +18,7 @@ function printLine($msg) {
 }
 CODE;
 
-$parser = (new ParserFactory())->createForHostVersion();
+$parser = (new ParserFactory)->create(ParserFactory::PREFER_PHP7);
 
 try {
     $stmts = $parser->parse($code);

doc/component/Lexer.markdown

@@ -42,17 +42,16 @@ The attributes used in this example match the default behavior of the lexer. The
 > **Note:** The example in this section is outdated in that this information is directly available in the AST: While
 > `$property->isPublic()` does not distinguish between `public` and `var`, directly checking `$property->flags` for
 > the `$property->flags & Class_::VISIBILITY_MODIFIER_MASK) === 0` allows making this distinction without resorting to
-> tokens. However, the general idea behind the example still applies in other cases.
+> tokens. However the general idea behind the example still applies in other cases.
 
 The token offset information is useful if you wish to examine the exact formatting used for a node. For example the AST
 does not distinguish whether a property was declared using `public` or using `var`, but you can retrieve this
 information based on the token position:
 
 ```php
-/** @param PhpParser\Token[] $tokens */
 function isDeclaredUsingVar(array $tokens, PhpParser\Node\Stmt\Property $prop) {
-    $i = $prop->getStartTokenPos();
-    return $tokens[$i]->id === T_VAR;
+    $i = $prop->getAttribute('startTokenPos');
+    return $tokens[$i][0] === T_VAR;
 }
 ```
 
@@ -73,12 +72,12 @@ class MyNodeVisitor extends PhpParser\NodeVisitorAbstract {
     }
 }
 
-$lexerOptions = array(
+$lexer = new PhpParser\Lexer(array(
     'usedAttributes' => array(
         'comments', 'startLine', 'endLine', 'startTokenPos', 'endTokenPos'
     )
-);
-$parser = (new PhpParser\ParserFactory())->createForHostVersion($lexerOptions);
+));
+$parser = (new PhpParser\ParserFactory)->create(PhpParser\ParserFactory::ONLY_PHP7, $lexer);
 
 $visitor = new MyNodeVisitor();
 $traverser = new PhpParser\NodeTraverser();
@@ -112,15 +111,14 @@ The `startLexing()` method is invoked whenever the `parse()` method of the parse
 code that is to be lexed (including the opening tag). It can be used to reset state or preprocess the source code or tokens. The
 passed `ErrorHandler` should be used to report lexing errors.
 
-The `getTokens()` method returns the current array of `PhpParser\Token`s, which are compatible with the PHP 8 `PhpToken`
-class. This method is not used by the parser (which uses `getNextToken()`), but is useful in combination with the token
-position attributes.
+The `getTokens()` method returns the current token array, in the usual `token_get_all()` format. This method is not
+used by the parser (which uses `getNextToken()`), but is useful in combination with the token position attributes.
 
 The `handleHaltCompiler()` method is called whenever a `T_HALT_COMPILER` token is encountered. It has to return the
 remaining string after the construct (not including `();`).
 
-The `getNextToken()` method returns the ID of the next token (in the sense of `Token::$id`). If no more
-tokens are available it must return `0`, which is the ID of the `EOF` token. Furthermore, the string content of the
+The `getNextToken()` method returns the ID of the next token (as defined by the `Parser::T_*` constants). If no more
+tokens are available it must return `0`, which is the ID of the `EOF` token. Furthermore the string content of the
 token should be written into the by-reference `$value` parameter (which will then be available as `$n` in the parser).
 
 ### Attribute handling
@@ -146,10 +144,10 @@ class KeepOriginalValueLexer extends Lexer // or Lexer\Emulative
     public function getNextToken(&$value = null, &$startAttributes = null, &$endAttributes = null) {
         $tokenId = parent::getNextToken($value, $startAttributes, $endAttributes);
 
-        if ($tokenId == \T_CONSTANT_ENCAPSED_STRING   // non-interpolated string
-            || $tokenId == \T_ENCAPSED_AND_WHITESPACE // interpolated string
-            || $tokenId == \T_LNUMBER                 // integer
-            || $tokenId == \T_DNUMBER                 // floating point number
+        if ($tokenId == Tokens::T_CONSTANT_ENCAPSED_STRING   // non-interpolated string
+            || $tokenId == Tokens::T_ENCAPSED_AND_WHITESPACE // interpolated string
+            || $tokenId == Tokens::T_LNUMBER                 // integer
+            || $tokenId == Tokens::T_DNUMBER                 // floating point number
         ) {
             // could also use $startAttributes, doesn't really matter here
             $endAttributes['originalValue'] = $value;

doc/component/Name_resolution.markdown

@@ -10,7 +10,7 @@ visitor (NameResolver) based on it.
 The NameResolver visitor
 ------------------------
 
-The `NameResolver` visitor can (and for nearly all uses of the AST, should) be applied to resolve names
+The `NameResolver` visitor can (and for nearly all uses of the AST, is) be applied to resolve names
 to their fully-qualified form, to the degree that this is possible.
 
 ```php
@@ -53,7 +53,7 @@ name. Once again, if an unqualified function or constant name cannot be resolved
 `resolvedName` attribute will not be present, and instead a `namespacedName` attribute is added.
 
 The `replaceNodes` attribute is useful if you wish to perform modifications on the AST, as you
-probably do not wish the resulting code to have fully resolved names as a side-effect.
+probably do not wish the resoluting code to have fully resolved names as a side-effect.
 
 The NameContext
 ---------------
@@ -84,4 +84,4 @@ representation of a name given the current name resolution environment.
 The name context is intended to be used for name resolution operations outside the AST itself, such
 as class names inside doc comments. A visitor running in parallel with the name resolver can access
 the name context using `$nameResolver->getNameContext()`. Alternatively a visitor can use an
-independent context and explicitly feed `Namespace` and `Use` nodes to it.
+independent context and explicitly feed `Namespace` and `Use` nodes to it.

doc/component/Pretty_printing.markdown

@@ -31,16 +31,10 @@ expression.
 Customizing the formatting
 --------------------------
 
-The pretty printer respects a number of `kind` attributes used by some notes (e.g., whether an
-integer should be printed as decimal, hexadecimal, etc). Additionally, it supports two options:
-
-* `phpVersion` (defaults to 7.0) allows opting into formatting that is not supported by older PHP
-  versions.
-* `shortArraySyntax` determines the used array syntax if the `kind` attribute is not set. This is
-  a legacy option, and `phpVersion` should be used to control this behavior instead.
-
-However, the default pretty printer does not provide any functionality for fine-grained
-customization of code formatting.
+Apart from an `shortArraySyntax` option, the default pretty printer does not provide any
+functionality to customize the formatting of the generated code. The pretty printer does respect a
+number of `kind` attributes used by some notes (e.g., whether an integer should be printed as
+decimal, hexadecimal, etc), but there are no options to control brace placement or similar.
 
 If you want to make minor changes to the formatting, the easiest way is to extend the pretty printer
 and override the methods responsible for the node types you are interested in.
@@ -52,27 +46,29 @@ default pretty printer with an existing library for code reformatting, such as
 Formatting-preserving pretty printing
 -------------------------------------
 
+> **Note:** This functionality is **experimental** and not yet complete.
+
 For automated code refactoring, migration and similar, you will usually only want to modify a small
 portion of the code and leave the remainder alone. The basic pretty printer is not suitable for
 this, because it will also reformat parts of the code which have not been modified.
 
-Since PHP-Parser 4.0, a formatting-preserving pretty-printing mode is available, which
+Since PHP-Parser 4.0, an experimental formatting-preserving pretty-printing mode is available, which
 attempts to preserve the formatting of code (those AST nodes that have not changed) and only reformat
 code which has been modified or newly inserted.
 
 Use of the formatting-preservation functionality requires some additional preparatory steps:
 
 ```php
-use PhpParser\{Lexer, NodeTraverser, NodeVisitor, ParserFactory, PrettyPrinter};
+use PhpParser\{Lexer, NodeTraverser, NodeVisitor, Parser, PrettyPrinter};
 
-$lexerOptions = new [
+$lexer = new Lexer\Emulative([
     'usedAttributes' => [
         'comments',
         'startLine', 'endLine',
         'startTokenPos', 'endTokenPos',
     ],
-];
-$parser = (new ParserFactory())->createForHostVersion($lexerOptions);
+]);
+$parser = new Parser\Php7($lexer);
 
 $traverser = new NodeTraverser();
 $traverser->addVisitor(new NodeVisitor\CloningVisitor());
@@ -80,7 +76,7 @@ $traverser->addVisitor(new NodeVisitor\CloningVisitor());
 $printer = new PrettyPrinter\Standard();
 
 $oldStmts = $parser->parse($code);
-$oldTokens = $parser->getLexer()->getTokens();
+$oldTokens = $lexer->getTokens();
 
 $newStmts = $traverser->traverse($oldStmts);
 
@@ -90,9 +86,11 @@ $newCode = $printer->printFormatPreserving($newStmts, $oldStmts, $oldTokens);
 ```
 
 If you make use of the name resolution functionality, you will likely want to disable the
-`replaceNodes` option. This will add resolved names as attributes, instead of directly modifying
+`replaceNodes` option. This will add resolved names as attributes, instead of directlying modifying
 the AST and causing spurious changes to the pretty printed code. For more information, see the
 [name resolution documentation](Name_resolution.markdown).
 
-The formatting-preservation works on a best-effort basis and may sometimes reformat more code tha
-necessary. If you encounter problems while using this functionality, please open an issue.
+This functionality is experimental and not yet fully implemented. It should not provide incorrect
+code, but it may sometimes reformat more code than necessary. Open issues are tracked in
+[issue #344](https://github.com/nikic/PHP-Parser/issues/344). If you encounter problems while using
+this functionality, please open an issue, so we know what to prioritize.

doc/component/Walking_the_AST.markdown

@@ -11,7 +11,7 @@ use PhpParser\{Node, NodeTraverser, NodeVisitorAbstract};
 $traverser = new NodeTraverser;
 $traverser->addVisitor(new class extends NodeVisitorAbstract {
     public function leaveNode(Node $node) {
-        if ($node instanceof Node\Scalar\Int_) {
+        if ($node instanceof Node\Scalar\LNumber) {
             return new Node\Scalar\String_((string) $node->value);
         }
     }
@@ -129,7 +129,8 @@ Now `$a && $b` will be replaced by `!($a && $b)`. Then the traverser will go int
 only) child of `!($a && $b)`, which is `$a && $b`. The transformation applies again and we end up
 with `!!($a && $b)`. This will continue until PHP hits the memory limit.
 
-Finally, there are two special replacement types. The first is removal of a node:
+Finally, two special replacement types are supported only by leaveNode. The first is removal of a
+node:
 
 ```php
 public function leaveNode(Node $node) {
@@ -164,8 +165,8 @@ This example will remove all calls to `var_dump()` which occur as expression sta
 that `var_dump($a);` will be removed, but `if (var_dump($a))` will not be removed (and there is no
 obvious way in which it can be removed).
 
-Next to removing nodes, it is also possible to replace one node with multiple nodes. This
-only works if the parent structure is an array.
+Next to removing nodes, it is also possible to replace one node with multiple nodes. Again, this
+only works inside leaveNode and only if the parent structure is an array.
 
 ```php
 public function leaveNode(Node $node) {

grammar/README.md

@@ -1,8 +1,11 @@
 What do all those files mean?
 =============================
 
- * `php.y`:              PHP 5-8 grammar written in a pseudo language
+ * `php5.y`:             PHP 5 grammar written in a pseudo language
+ * `php7.y`:             PHP 7 grammar written in a pseudo language
+ * `tokens.y`:           Tokens definition shared between PHP 5 and PHP 7 grammars
  * `parser.template`:    A `kmyacc` parser prototype file for PHP
+ * `tokens.template`:    A `kmyacc` prototype file for the `Tokens` class
  * `rebuildParsers.php`: Preprocesses the grammar and builds the parser using `kmyacc`
 
 .phpy pseudo language

grammar/parser.template

@@ -1,4 +1,4 @@
-<?php declare(strict_types=1);
+<?php
 $meta #
 #semval($) $this->semValue
 #semval($,%t) $this->semValue
@@ -8,7 +8,6 @@ $meta #
 namespace PhpParser\Parser;
 
 use PhpParser\Error;
-use PhpParser\Modifiers;
 use PhpParser\Node;
 use PhpParser\Node\Expr;
 use PhpParser\Node\Name;
@@ -18,16 +17,12 @@ use PhpParser\Node\Stmt;
 
 /* This is an automatically GENERATED file, which should not be manually edited.
  * Instead edit one of the following:
- *  * the grammar file grammar/php.y
+ *  * the grammar files grammar/php5.y or grammar/php7.y
  *  * the skeleton file grammar/parser.template
  *  * the preprocessing script grammar/rebuildParsers.php
  */
 class #(-p) extends \PhpParser\ParserAbstract
 {
-#tokenval
-    public const %s = %n;
-#endtokenval
-
     protected $tokenToSymbolMapSize = #(YYMAXLEX);
     protected $actionTableSize = #(YYLAST);
     protected $gotoTableSize = #(YYGLAST);
@@ -94,7 +89,7 @@ class #(-p) extends \PhpParser\ParserAbstract
     );
 #endif
 
-    protected function initReduceCallbacks(): void {
+    protected function initReduceCallbacks() {
         $this->reduceCallbacks = [
 #reduce
             %n => function ($stackPos) {

grammar/php7.y

@@ -1,125 +1,7 @@
 %pure_parser
 %expect 2
 
-%right T_THROW
-%left T_INCLUDE T_INCLUDE_ONCE T_EVAL T_REQUIRE T_REQUIRE_ONCE
-%left ','
-%left T_LOGICAL_OR
-%left T_LOGICAL_XOR
-%left T_LOGICAL_AND
-%right T_PRINT
-%right T_YIELD
-%right T_DOUBLE_ARROW
-%right T_YIELD_FROM
-%left '=' T_PLUS_EQUAL T_MINUS_EQUAL T_MUL_EQUAL T_DIV_EQUAL T_CONCAT_EQUAL T_MOD_EQUAL T_AND_EQUAL T_OR_EQUAL T_XOR_EQUAL T_SL_EQUAL T_SR_EQUAL T_POW_EQUAL T_COALESCE_EQUAL
-%left '?' ':'
-%right T_COALESCE
-%left T_BOOLEAN_OR
-%left T_BOOLEAN_AND
-%left '|'
-%left '^'
-%left T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG
-%nonassoc T_IS_EQUAL T_IS_NOT_EQUAL T_IS_IDENTICAL T_IS_NOT_IDENTICAL T_SPACESHIP
-%nonassoc '<' T_IS_SMALLER_OR_EQUAL '>' T_IS_GREATER_OR_EQUAL
-#if PHP7
-%left T_SL T_SR
-%left '+' '-' '.'
-#endif
-#if PHP8
-%left '.'
-%left T_SL T_SR
-%left '+' '-'
-#endif
-%left '*' '/' '%'
-%right '!'
-%nonassoc T_INSTANCEOF
-%right '~' T_INC T_DEC T_INT_CAST T_DOUBLE_CAST T_STRING_CAST T_ARRAY_CAST T_OBJECT_CAST T_BOOL_CAST T_UNSET_CAST '@'
-%right T_POW
-%right '['
-%nonassoc T_NEW T_CLONE
-%token T_EXIT
-%token T_IF
-%left T_ELSEIF
-%left T_ELSE
-%left T_ENDIF
-%token T_LNUMBER
-%token T_DNUMBER
-%token T_STRING
-%token T_STRING_VARNAME
-%token T_VARIABLE
-%token T_NUM_STRING
-%token T_INLINE_HTML
-%token T_ENCAPSED_AND_WHITESPACE
-%token T_CONSTANT_ENCAPSED_STRING
-%token T_ECHO
-%token T_DO
-%token T_WHILE
-%token T_ENDWHILE
-%token T_FOR
-%token T_ENDFOR
-%token T_FOREACH
-%token T_ENDFOREACH
-%token T_DECLARE
-%token T_ENDDECLARE
-%token T_AS
-%token T_SWITCH
-%token T_MATCH
-%token T_ENDSWITCH
-%token T_CASE
-%token T_DEFAULT
-%token T_BREAK
-%token T_CONTINUE
-%token T_GOTO
-%token T_FUNCTION
-%token T_FN
-%token T_CONST
-%token T_RETURN
-%token T_TRY
-%token T_CATCH
-%token T_FINALLY
-%token T_THROW
-%token T_USE
-%token T_INSTEADOF
-%token T_GLOBAL
-%right T_STATIC T_ABSTRACT T_FINAL T_PRIVATE T_PROTECTED T_PUBLIC T_READONLY
-%token T_VAR
-%token T_UNSET
-%token T_ISSET
-%token T_EMPTY
-%token T_HALT_COMPILER
-%token T_CLASS
-%token T_TRAIT
-%token T_INTERFACE
-%token T_ENUM
-%token T_EXTENDS
-%token T_IMPLEMENTS
-%token T_OBJECT_OPERATOR
-%token T_NULLSAFE_OBJECT_OPERATOR
-%token T_DOUBLE_ARROW
-%token T_LIST
-%token T_ARRAY
-%token T_CALLABLE
-%token T_CLASS_C
-%token T_TRAIT_C
-%token T_METHOD_C
-%token T_FUNC_C
-%token T_LINE
-%token T_FILE
-%token T_START_HEREDOC
-%token T_END_HEREDOC
-%token T_DOLLAR_OPEN_CURLY_BRACES
-%token T_CURLY_OPEN
-%token T_PAAMAYIM_NEKUDOTAYIM
-%token T_NAMESPACE
-%token T_NS_C
-%token T_DIR
-%token T_NS_SEPARATOR
-%token T_ELLIPSIS
-%token T_NAME_FULLY_QUALIFIED
-%token T_NAME_QUALIFIED
-%token T_NAME_RELATIVE
-%token T_ATTRIBUTE
-%token T_ENUM
+%tokens
 
 %%
 
@@ -145,14 +27,13 @@ ampersand:
 
 reserved_non_modifiers:
       T_INCLUDE | T_INCLUDE_ONCE | T_EVAL | T_REQUIRE | T_REQUIRE_ONCE | T_LOGICAL_OR | T_LOGICAL_XOR | T_LOGICAL_AND
-    | T_INSTANCEOF | T_NEW | T_CLONE | T_EXIT | T_IF | T_ELSEIF | T_ELSE | T_ENDIF | T_DO | T_WHILE
+    | T_INSTANCEOF | T_NEW | T_CLONE | T_EXIT | T_IF | T_ELSEIF | T_ELSE | T_ENDIF | T_ECHO | T_DO | T_WHILE
     | T_ENDWHILE | T_FOR | T_ENDFOR | T_FOREACH | T_ENDFOREACH | T_DECLARE | T_ENDDECLARE | T_AS | T_TRY | T_CATCH
     | T_FINALLY | T_THROW | T_USE | T_INSTEADOF | T_GLOBAL | T_VAR | T_UNSET | T_ISSET | T_EMPTY | T_CONTINUE | T_GOTO
     | T_FUNCTION | T_CONST | T_RETURN | T_PRINT | T_YIELD | T_LIST | T_SWITCH | T_ENDSWITCH | T_CASE | T_DEFAULT
     | T_BREAK | T_ARRAY | T_CALLABLE | T_EXTENDS | T_IMPLEMENTS | T_NAMESPACE | T_TRAIT | T_INTERFACE | T_CLASS
-    | T_CLASS_C | T_TRAIT_C | T_FUNC_C | T_METHOD_C | T_LINE | T_FILE | T_DIR | T_NS_C | T_FN
+    | T_CLASS_C | T_TRAIT_C | T_FUNC_C | T_METHOD_C | T_LINE | T_FILE | T_DIR | T_NS_C | T_HALT_COMPILER | T_FN
     | T_MATCH | T_ENUM
-    | T_ECHO { $$ = $1; if ($$ === "<?=") $this->emitError(new Error('Cannot use "<?=" as an identifier', attributes())); }
 ;
 
 semi_reserved:
@@ -236,7 +117,7 @@ top_statement:
       statement                                             { $$ = $1; }
     | function_declaration_statement                        { $$ = $1; }
     | class_declaration_statement                           { $$ = $1; }
-    | T_HALT_COMPILER '(' ')' ';'
+    | T_HALT_COMPILER
           { $$ = Stmt\HaltCompiler[$this->lexer->handleHaltCompiler()]; }
     | T_NAMESPACE namespace_declaration_name semi
           { $$ = Stmt\Namespace_[$2, null];
@@ -299,16 +180,16 @@ non_empty_inline_use_declarations:
 
 unprefixed_use_declaration:
       namespace_name
-          { $$ = Node\UseItem[$1, null, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #1); }
+          { $$ = Stmt\UseUse[$1, null, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #1); }
     | namespace_name T_AS identifier_not_reserved
-          { $$ = Node\UseItem[$1, $3, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #3); }
+          { $$ = Stmt\UseUse[$1, $3, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #3); }
 ;
 
 use_declaration:
       legacy_namespace_name
-          { $$ = Node\UseItem[$1, null, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #1); }
+          { $$ = Stmt\UseUse[$1, null, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #1); }
     | legacy_namespace_name T_AS identifier_not_reserved
-          { $$ = Node\UseItem[$1, $3, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #3); }
+          { $$ = Stmt\UseUse[$1, $3, Stmt\Use_::TYPE_UNKNOWN]; $this->checkUseUse($$, #3); }
 ;
 
 inline_use_declaration:
@@ -340,7 +221,10 @@ non_empty_class_const_list:
 ;
 
 class_const:
-    identifier_maybe_reserved '=' expr                      { $$ = Node\Const_[$1, $3]; }
+      T_STRING '=' expr
+          { $$ = Node\Const_[new Node\Identifier($1, stackAttributes(#1)), $3]; }
+    | semi_reserved '=' expr
+          { $$ = Node\Const_[new Node\Identifier($1, stackAttributes(#1)), $3]; }
 ;
 
 inner_statement_list_ex:
@@ -518,9 +402,9 @@ class_modifiers:
 ;
 
 class_modifier:
-      T_ABSTRACT                                            { $$ = Modifiers::ABSTRACT; }
-    | T_FINAL                                               { $$ = Modifiers::FINAL; }
-    | T_READONLY                                            { $$ = Modifiers::READONLY; }
+      T_ABSTRACT                                            { $$ = Stmt\Class_::MODIFIER_ABSTRACT; }
+    | T_FINAL                                               { $$ = Stmt\Class_::MODIFIER_FINAL; }
+    | T_READONLY                                            { $$ = Stmt\Class_::MODIFIER_READONLY; }
 ;
 
 extends_from:
@@ -573,7 +457,7 @@ non_empty_declare_list:
 ;
 
 declare_list_element:
-      identifier_not_reserved '=' expr                      { $$ = Node\DeclareItem[$1, $3]; }
+      identifier_not_reserved '=' expr                      { $$ = Stmt\DeclareDeclare[$1, $3]; }
 ;
 
 switch_case_list:
@@ -656,8 +540,7 @@ foreach_variable:
       variable                                              { $$ = array($1, false); }
     | ampersand variable                                    { $$ = array($2, true); }
     | list_expr                                             { $$ = array($1, false); }
-    | array_short_syntax
-          { $$ = array($this->fixupArrayDestructuring($1), false); }
+    | array_short_syntax                                    { $$ = array($1, false); }
 ;
 
 parameter_list:
@@ -677,10 +560,10 @@ optional_property_modifiers:
 ;
 
 property_modifier:
-      T_PUBLIC                  { $$ = Modifiers::PUBLIC; }
-    | T_PROTECTED               { $$ = Modifiers::PROTECTED; }
-    | T_PRIVATE                 { $$ = Modifiers::PRIVATE; }
-    | T_READONLY                { $$ = Modifiers::READONLY; }
+      T_PUBLIC                  { $$ = Stmt\Class_::MODIFIER_PUBLIC; }
+    | T_PROTECTED               { $$ = Stmt\Class_::MODIFIER_PROTECTED; }
+    | T_PRIVATE                 { $$ = Stmt\Class_::MODIFIER_PRIVATE; }
+    | T_READONLY                { $$ = Stmt\Class_::MODIFIER_READONLY; }
 ;
 
 parameter:
@@ -820,12 +703,12 @@ non_empty_static_var_list:
 ;
 
 static_var:
-      plain_variable                                        { $$ = Node\StaticVar[$1, null]; }
-    | plain_variable '=' expr                               { $$ = Node\StaticVar[$1, $3]; }
+      plain_variable                                        { $$ = Stmt\StaticVar[$1, null]; }
+    | plain_variable '=' expr                               { $$ = Stmt\StaticVar[$1, $3]; }
 ;
 
 class_statement_list_ex:
-      class_statement_list_ex class_statement               { if ($2 !== null) { push($1, $2); } }
+      class_statement_list_ex class_statement               { if ($2 !== null) { push($1, $2); } else { $$ = $1; } }
     | /* empty */                                           { init(); }
 ;
 
@@ -842,6 +725,9 @@ class_statement:
     | optional_attributes method_modifiers T_CONST class_const_list semi
           { $$ = new Stmt\ClassConst($4, $2, attributes(), $1);
             $this->checkClassConst($$, #2); }
+    | optional_attributes method_modifiers T_CONST type_expr class_const_list semi
+          { $$ = new Stmt\ClassConst($5, $2, attributes(), $1, $4);
+            $this->checkClassConst($$, #2); }
     | optional_attributes method_modifiers T_FUNCTION optional_ref identifier_maybe_reserved '(' parameter_list ')'
       optional_return_type method_body
           { $$ = Stmt\ClassMethod[$5, ['type' => $2, 'byRef' => $4, 'params' => $7, 'returnType' => $9, 'stmts' => $10, 'attrGroups' => $1]];
@@ -904,13 +790,13 @@ non_empty_member_modifiers:
 ;
 
 member_modifier:
-      T_PUBLIC                                              { $$ = Modifiers::PUBLIC; }
-    | T_PROTECTED                                           { $$ = Modifiers::PROTECTED; }
-    | T_PRIVATE                                             { $$ = Modifiers::PRIVATE; }
-    | T_STATIC                                              { $$ = Modifiers::STATIC; }
-    | T_ABSTRACT                                            { $$ = Modifiers::ABSTRACT; }
-    | T_FINAL                                               { $$ = Modifiers::FINAL; }
-    | T_READONLY                                            { $$ = Modifiers::READONLY; }
+      T_PUBLIC                                              { $$ = Stmt\Class_::MODIFIER_PUBLIC; }
+    | T_PROTECTED                                           { $$ = Stmt\Class_::MODIFIER_PROTECTED; }
+    | T_PRIVATE                                             { $$ = Stmt\Class_::MODIFIER_PRIVATE; }
+    | T_STATIC                                              { $$ = Stmt\Class_::MODIFIER_STATIC; }
+    | T_ABSTRACT                                            { $$ = Stmt\Class_::MODIFIER_ABSTRACT; }
+    | T_FINAL                                               { $$ = Stmt\Class_::MODIFIER_FINAL; }
+    | T_READONLY                                            { $$ = Stmt\Class_::MODIFIER_READONLY; }
 ;
 
 property_declaration_list:
@@ -928,8 +814,8 @@ property_decl_name:
 ;
 
 property_declaration:
-      property_decl_name                                    { $$ = Node\PropertyItem[$1, null]; }
-    | property_decl_name '=' expr                           { $$ = Node\PropertyItem[$1, $3]; }
+      property_decl_name                                    { $$ = Stmt\PropertyProperty[$1, null]; }
+    | property_decl_name '=' expr                           { $$ = Stmt\PropertyProperty[$1, $3]; }
 ;
 
 expr_list_forbid_comma:
@@ -953,16 +839,9 @@ for_expr:
 expr:
       variable                                              { $$ = $1; }
     | list_expr '=' expr                                    { $$ = Expr\Assign[$1, $3]; }
-    | array_short_syntax '=' expr
-          { $$ = Expr\Assign[$this->fixupArrayDestructuring($1), $3]; }
+    | array_short_syntax '=' expr                           { $$ = Expr\Assign[$1, $3]; }
     | variable '=' expr                                     { $$ = Expr\Assign[$1, $3]; }
     | variable '=' ampersand variable                       { $$ = Expr\AssignRef[$1, $4]; }
-    | variable '=' ampersand new_expr
-          { $$ = Expr\AssignRef[$1, $4];
-            if (!$this->phpVersion->allowsAssignNewByReference()) {
-                $this->emitError(new Error('Cannot assign new by reference', attributes()));
-            }
-          }
     | new_expr                                              { $$ = $1; }
     | match                                                 { $$ = $1; }
     | T_CLONE expr                                          { $$ = Expr\Clone_[$2]; }
@@ -1070,8 +949,8 @@ expr:
 ;
 
 anonymous_class:
-      optional_attributes T_CLASS ctor_arguments extends_from implements_list '{' class_statement_list '}'
-          { $$ = array(Stmt\Class_[null, ['type' => 0, 'extends' => $4, 'implements' => $5, 'stmts' => $7, 'attrGroups' => $1]], $3);
+      optional_attributes class_entry_type ctor_arguments extends_from implements_list '{' class_statement_list '}'
+          { $$ = array(Stmt\Class_[null, ['type' => $2, 'extends' => $4, 'implements' => $5, 'stmts' => $7, 'attrGroups' => $1]], $3);
             $this->checkClass($$[0], -1); }
 ;
 
@@ -1096,7 +975,7 @@ non_empty_lexical_var_list:
 ;
 
 lexical_var:
-      optional_ref plain_variable                           { $$ = Node\ClosureUse[$2, $1]; }
+      optional_ref plain_variable                           { $$ = Expr\ClosureUse[$2, $1]; }
 ;
 
 name_readonly:
@@ -1143,7 +1022,7 @@ exit_expr:
 backticks_expr:
       /* empty */                                           { $$ = array(); }
     | T_ENCAPSED_AND_WHITESPACE
-          { $$ = array(Node\InterpolatedStringPart[Scalar\String_::parseEscapeSequences($1, '`')]); }
+          { $$ = array(Scalar\EncapsedStringPart[Scalar\String_::parseEscapeSequences($1, '`')]); }
     | encaps_list                                           { parseEncapsed($1, '`', true); $$ = $1; }
 ;
 
@@ -1184,19 +1063,17 @@ array_short_syntax:
 dereferencable_scalar:
       T_ARRAY '(' array_pair_list ')'
           { $attrs = attributes(); $attrs['kind'] = Expr\Array_::KIND_LONG;
-            $$ = new Expr\Array_($3, $attrs);
-            $this->createdArrays->attach($$); }
-    | array_short_syntax                                    { $$ = $1; $this->createdArrays->attach($$); }
+            $$ = new Expr\Array_($3, $attrs); }
+    | array_short_syntax                                    { $$ = $1; }
     | T_CONSTANT_ENCAPSED_STRING                            { $$ = Scalar\String_::fromString($1, attributes()); }
     | '"' encaps_list '"'
           { $attrs = attributes(); $attrs['kind'] = Scalar\String_::KIND_DOUBLE_QUOTED;
-            parseEncapsed($2, '"', true); $$ = new Scalar\InterpolatedString($2, $attrs); }
+            parseEncapsed($2, '"', true); $$ = new Scalar\Encapsed($2, $attrs); }
 ;
 
 scalar:
-      T_LNUMBER
-          { $$ = $this->parseLNumber($1, attributes(), $this->phpVersion->allowsInvalidOctals()); }
-    | T_DNUMBER                                             { $$ = Scalar\Float_::fromString($1, attributes()); }
+      T_LNUMBER                                             { $$ = $this->parseLNumber($1, attributes()); }
+    | T_DNUMBER                                             { $$ = Scalar\DNumber::fromString($1, attributes()); }
     | dereferencable_scalar                                 { $$ = $1; }
     | constant                                              { $$ = $1; }
     | class_constant                                        { $$ = $1; }
@@ -1299,14 +1176,12 @@ property_name:
 ;
 
 list_expr:
-      T_LIST '(' inner_array_pair_list ')'
-          { $$ = Expr\List_[$3]; $$->setAttribute('kind', Expr\List_::KIND_LIST);
-            $this->postprocessList($$); }
+      T_LIST '(' inner_array_pair_list ')'                  { $$ = Expr\List_[$3]; }
 ;
 
 array_pair_list:
       inner_array_pair_list
-          { $$ = $1; $end = count($$)-1; if ($$[$end]->value instanceof Expr\Error) array_pop($$); }
+          { $$ = $1; $end = count($$)-1; if ($$[$end] === null) array_pop($$); }
 ;
 
 comma_or_error:
@@ -1321,18 +1196,14 @@ inner_array_pair_list:
 ;
 
 array_pair:
-      expr                                                  { $$ = Node\ArrayItem[$1, null, false]; }
-    | ampersand variable                                    { $$ = Node\ArrayItem[$2, null, true]; }
-    | list_expr                                             { $$ = Node\ArrayItem[$1, null, false]; }
-    | expr T_DOUBLE_ARROW expr                              { $$ = Node\ArrayItem[$3, $1,   false]; }
-    | expr T_DOUBLE_ARROW ampersand variable                { $$ = Node\ArrayItem[$4, $1,   true]; }
-    | expr T_DOUBLE_ARROW list_expr                         { $$ = Node\ArrayItem[$3, $1,   false]; }
-    | T_ELLIPSIS expr                                       { $$ = new Node\ArrayItem($2, null, false, attributes(), true); }
-    | /* empty */
-        { /* Create an Error node now to remember the position. We'll later either report an error,
-             or convert this into a null element, depending on whether this is a creation or destructuring context. */
-          $attrs = $this->createEmptyElemAttributes($this->lookaheadStartAttributes);
-          $$ = new Node\ArrayItem(new Expr\Error($attrs), null, false, $attrs); }
+      expr                                                  { $$ = Expr\ArrayItem[$1, null, false]; }
+    | ampersand variable                                    { $$ = Expr\ArrayItem[$2, null, true]; }
+    | list_expr                                             { $$ = Expr\ArrayItem[$1, null, false]; }
+    | expr T_DOUBLE_ARROW expr                              { $$ = Expr\ArrayItem[$3, $1,   false]; }
+    | expr T_DOUBLE_ARROW ampersand variable                { $$ = Expr\ArrayItem[$4, $1,   true]; }
+    | expr T_DOUBLE_ARROW list_expr                         { $$ = Expr\ArrayItem[$3, $1,   false]; }
+    | T_ELLIPSIS expr                                       { $$ = new Expr\ArrayItem($2, null, false, attributes(), true); }
+    | /* empty */                                           { $$ = null; }
 ;
 
 encaps_list:
@@ -1343,7 +1214,7 @@ encaps_list:
 ;
 
 encaps_string_part:
-      T_ENCAPSED_AND_WHITESPACE                             { $$ = Node\InterpolatedStringPart[$1]; }
+      T_ENCAPSED_AND_WHITESPACE                             { $$ = Scalar\EncapsedStringPart[$1]; }
 ;
 
 encaps_str_varname:

grammar/phpyLang.php

@@ -1,4 +1,4 @@
-<?php declare(strict_types=1);
+<?php
 
 ///////////////////////////////
 /// Utility regex constants ///
@@ -30,7 +30,7 @@ function preprocessGrammar($code) {
 function resolveNodes($code) {
     return preg_replace_callback(
         '~\b(?<name>[A-Z][a-zA-Z_\\\\]++)\s*' . PARAMS . '~',
-        function ($matches) {
+        function($matches) {
             // recurse
             $matches['params'] = resolveNodes($matches['params']);
 
@@ -53,7 +53,7 @@ function resolveNodes($code) {
 function resolveMacros($code) {
     return preg_replace_callback(
         '~\b(?<!::|->)(?!array\()(?<name>[a-z][A-Za-z]++)' . ARGS . '~',
-        function ($matches) {
+        function($matches) {
             // recurse
             $matches['args'] = resolveMacros($matches['args']);
 
@@ -106,7 +106,7 @@ function resolveMacros($code) {
             if ('parseEncapsed' === $name) {
                 assertArgs(3, $args, $name);
 
-                return 'foreach (' . $args[0] . ' as $s) { if ($s instanceof Node\InterpolatedStringPart) {'
+                return 'foreach (' . $args[0] . ' as $s) { if ($s instanceof Node\Scalar\EncapsedStringPart) {'
                        . ' $s->value = Node\Scalar\String_::parseEscapeSequences($s->value, ' . $args[1] . ', ' . $args[2] . '); } }';
             }
 

grammar/rebuildParsers.php

@@ -1,17 +1,19 @@
-<?php declare(strict_types=1);
+<?php
 
 require __DIR__ . '/phpyLang.php';
 
-$parserToDefines = [
-    'Php7' => ['PHP7' => true],
-    'Php8' => ['PHP8' => true],
+$grammarFileToName = [
+    __DIR__ . '/php5.y' => 'Php5',
+    __DIR__ . '/php7.y' => 'Php7',
 ];
 
-$grammarFile    = __DIR__ . '/php.y';
+$tokensFile     = __DIR__ . '/tokens.y';
+$tokensTemplate = __DIR__ . '/tokens.template';
 $skeletonFile   = __DIR__ . '/parser.template';
 $tmpGrammarFile = __DIR__ . '/tmp_parser.phpy';
 $tmpResultFile  = __DIR__ . '/tmp_parser.php';
 $resultDir = __DIR__ . '/../lib/PhpParser/Parser';
+$tokensResultsFile = $resultDir . '/Tokens.php';
 
 $kmyacc = getenv('KMYACC');
 if (!$kmyacc) {
@@ -27,11 +29,13 @@ $optionKeepTmpGrammar = isset($options['--keep-tmp-grammar']);
 /// Main script ///
 ///////////////////
 
-foreach ($parserToDefines as $name => $defines) {
+$tokens = file_get_contents($tokensFile);
+
+foreach ($grammarFileToName as $grammarFile => $name) {
     echo "Building temporary $name grammar file.\n";
 
     $grammarCode = file_get_contents($grammarFile);
-    $grammarCode = replaceIfBlocks($grammarCode, $defines);
+    $grammarCode = str_replace('%tokens', $tokens, $grammarCode);
     $grammarCode = preprocessGrammar($grammarCode);
 
     file_put_contents($tmpGrammarFile, $grammarCode);
@@ -48,6 +52,10 @@ foreach ($parserToDefines as $name => $defines) {
     file_put_contents("$resultDir/$name.php", $resultCode);
     unlink($tmpResultFile);
 
+    echo "Building token definition.\n";
+    $output = execCmd("$kmyacc -m $tokensTemplate $tmpGrammarFile");
+    rename($tmpResultFile, $tokensResultsFile);
+
     if (!$optionKeepTmpGrammar) {
         unlink($tmpGrammarFile);
     }
@@ -64,17 +72,10 @@ function ensureDirExists($dir) {
 }
 
 function execCmd($cmd) {
-    $output = trim(shell_exec("$cmd 2>&1") ?? '');
+    $output = trim(shell_exec("$cmd 2>&1"));
     if ($output !== "") {
         echo "> " . $cmd . "\n";
         echo $output;
     }
     return $output;
 }
-
-function replaceIfBlocks(string $code, array $defines): string {
-    return preg_replace_callback('/\n#if\s+(\w+)\n(.*?)\n#endif/s', function ($matches) use ($defines) {
-        $value = $defines[$matches[1]] ?? false;
-        return $value ? $matches[2] : '';
-    }, $code);
-}

grammar/tokens.template

@@ -0,0 +1,17 @@
+<?php
+$meta #
+#semval($) $this->semValue
+#semval($,%t) $this->semValue
+#semval(%n) $this->stackPos-(%l-%n)
+#semval(%n,%t) $this->stackPos-(%l-%n)
+
+namespace PhpParser\Parser;
+#include;
+
+/* GENERATED file based on grammar/tokens.y */
+final class Tokens
+{
+#tokenval
+    const %s = %n;
+#endtokenval
+}

grammar/tokens.y

@@ -0,0 +1,115 @@
+/* We currently rely on the token ID mapping to be the same between PHP 5 and PHP 7 - so the same lexer can be used for
+ * both. This is enforced by sharing this token file. */
+
+%right T_THROW
+%left T_INCLUDE T_INCLUDE_ONCE T_EVAL T_REQUIRE T_REQUIRE_ONCE
+%left ','
+%left T_LOGICAL_OR
+%left T_LOGICAL_XOR
+%left T_LOGICAL_AND
+%right T_PRINT
+%right T_YIELD
+%right T_DOUBLE_ARROW
+%right T_YIELD_FROM
+%left '=' T_PLUS_EQUAL T_MINUS_EQUAL T_MUL_EQUAL T_DIV_EQUAL T_CONCAT_EQUAL T_MOD_EQUAL T_AND_EQUAL T_OR_EQUAL T_XOR_EQUAL T_SL_EQUAL T_SR_EQUAL T_POW_EQUAL T_COALESCE_EQUAL
+%left '?' ':'
+%right T_COALESCE
+%left T_BOOLEAN_OR
+%left T_BOOLEAN_AND
+%left '|'
+%left '^'
+%left T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG
+%nonassoc T_IS_EQUAL T_IS_NOT_EQUAL T_IS_IDENTICAL T_IS_NOT_IDENTICAL T_SPACESHIP
+%nonassoc '<' T_IS_SMALLER_OR_EQUAL '>' T_IS_GREATER_OR_EQUAL
+%left T_SL T_SR
+%left '+' '-' '.'
+%left '*' '/' '%'
+%right '!'
+%nonassoc T_INSTANCEOF
+%right '~' T_INC T_DEC T_INT_CAST T_DOUBLE_CAST T_STRING_CAST T_ARRAY_CAST T_OBJECT_CAST T_BOOL_CAST T_UNSET_CAST '@'
+%right T_POW
+%right '['
+%nonassoc T_NEW T_CLONE
+%token T_EXIT
+%token T_IF
+%left T_ELSEIF
+%left T_ELSE
+%left T_ENDIF
+%token T_LNUMBER
+%token T_DNUMBER
+%token T_STRING
+%token T_STRING_VARNAME
+%token T_VARIABLE
+%token T_NUM_STRING
+%token T_INLINE_HTML
+%token T_ENCAPSED_AND_WHITESPACE
+%token T_CONSTANT_ENCAPSED_STRING
+%token T_ECHO
+%token T_DO
+%token T_WHILE
+%token T_ENDWHILE
+%token T_FOR
+%token T_ENDFOR
+%token T_FOREACH
+%token T_ENDFOREACH
+%token T_DECLARE
+%token T_ENDDECLARE
+%token T_AS
+%token T_SWITCH
+%token T_MATCH
+%token T_ENDSWITCH
+%token T_CASE
+%token T_DEFAULT
+%token T_BREAK
+%token T_CONTINUE
+%token T_GOTO
+%token T_FUNCTION
+%token T_FN
+%token T_CONST
+%token T_RETURN
+%token T_TRY
+%token T_CATCH
+%token T_FINALLY
+%token T_THROW
+%token T_USE
+%token T_INSTEADOF
+%token T_GLOBAL
+%right T_STATIC T_ABSTRACT T_FINAL T_PRIVATE T_PROTECTED T_PUBLIC T_READONLY
+%token T_VAR
+%token T_UNSET
+%token T_ISSET
+%token T_EMPTY
+%token T_HALT_COMPILER
+%token T_CLASS
+%token T_TRAIT
+%token T_INTERFACE
+%token T_ENUM
+%token T_EXTENDS
+%token T_IMPLEMENTS
+%token T_OBJECT_OPERATOR
+%token T_NULLSAFE_OBJECT_OPERATOR
+%token T_DOUBLE_ARROW
+%token T_LIST
+%token T_ARRAY
+%token T_CALLABLE
+%token T_CLASS_C
+%token T_TRAIT_C
+%token T_METHOD_C
+%token T_FUNC_C
+%token T_LINE
+%token T_FILE
+%token T_START_HEREDOC
+%token T_END_HEREDOC
+%token T_DOLLAR_OPEN_CURLY_BRACES
+%token T_CURLY_OPEN
+%token T_PAAMAYIM_NEKUDOTAYIM
+%token T_NAMESPACE
+%token T_NS_C
+%token T_DIR
+%token T_NS_SEPARATOR
+%token T_ELLIPSIS
+%token T_NAME_FULLY_QUALIFIED
+%token T_NAME_QUALIFIED
+%token T_NAME_RELATIVE
+%token T_ATTRIBUTE
+%token T_ENUM

lib/PhpParser/Builder.php

@@ -2,11 +2,12 @@
 
 namespace PhpParser;
 
-interface Builder {
+interface Builder
+{
     /**
      * Returns the built node.
      *
      * @return Node The built node
      */
-    public function getNode(): Node;
+    public function getNode() : Node;
 }

lib/PhpParser/Builder/ClassConst.php

@@ -6,22 +6,21 @@ namespace PhpParser\Builder;
 
 use PhpParser;
 use PhpParser\BuilderHelpers;
-use PhpParser\Modifiers;
 use PhpParser\Node;
 use PhpParser\Node\Const_;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Stmt;
 
-class ClassConst implements PhpParser\Builder {
-    /** @var int */
+class ClassConst implements PhpParser\Builder
+{
     protected $flags = 0;
-    /** @var array<string, mixed> */
     protected $attributes = [];
-    /** @var list<Const_> */
     protected $constants = [];
 
-    /** @var list<Node\AttributeGroup> */
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
+    /** @var Identifier|Node\Name|Node\ComplexType */
+    protected $type;
 
     /**
      * Creates a class constant builder
@@ -53,7 +52,7 @@ class ClassConst implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makePublic() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PUBLIC);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PUBLIC);
 
         return $this;
     }
@@ -64,7 +63,7 @@ class ClassConst implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeProtected() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PROTECTED);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PROTECTED);
 
         return $this;
     }
@@ -75,7 +74,7 @@ class ClassConst implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makePrivate() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PRIVATE);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PRIVATE);
 
         return $this;
     }
@@ -86,7 +85,7 @@ class ClassConst implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeFinal() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::FINAL);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_FINAL);
 
         return $this;
     }
@@ -119,6 +118,19 @@ class ClassConst implements PhpParser\Builder {
         return $this;
     }
 
+    /**
+     * Sets the constant type.
+     *
+     * @param string|Node\Name|Identifier|Node\ComplexType $type
+     *
+     * @return $this
+     */
+    public function setType($type) {
+        $this->type = BuilderHelpers::normalizeType($type);
+
+        return $this;
+    }
+
     /**
      * Returns the built class node.
      *
@@ -129,7 +141,8 @@ class ClassConst implements PhpParser\Builder {
             $this->constants,
             $this->flags,
             $this->attributes,
-            $this->attributeGroups
+            $this->attributeGroups,
+            $this->type
         );
     }
 }

lib/PhpParser/Builder/Class_.php

@@ -4,31 +4,24 @@ namespace PhpParser\Builder;
 
 use PhpParser;
 use PhpParser\BuilderHelpers;
-use PhpParser\Modifiers;
 use PhpParser\Node;
 use PhpParser\Node\Name;
 use PhpParser\Node\Stmt;
 
-class Class_ extends Declaration {
-    /** @var string */
+class Class_ extends Declaration
+{
     protected $name;
 
-    /** @var Name|null */
     protected $extends = null;
-    /** @var list<Name> */
     protected $implements = [];
-    /** @var int */
     protected $flags = 0;
 
-    /** @var list<Stmt\TraitUse> */
     protected $uses = [];
-    /** @var list<Stmt\ClassConst> */
     protected $constants = [];
-    /** @var list<Stmt\Property> */
     protected $properties = [];
-    /** @var list<Stmt\ClassMethod> */
     protected $methods = [];
-    /** @var list<Node\AttributeGroup> */
+
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -74,7 +67,7 @@ class Class_ extends Declaration {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeAbstract() {
-        $this->flags = BuilderHelpers::addClassModifier($this->flags, Modifiers::ABSTRACT);
+        $this->flags = BuilderHelpers::addClassModifier($this->flags, Stmt\Class_::MODIFIER_ABSTRACT);
 
         return $this;
     }
@@ -85,18 +78,13 @@ class Class_ extends Declaration {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeFinal() {
-        $this->flags = BuilderHelpers::addClassModifier($this->flags, Modifiers::FINAL);
+        $this->flags = BuilderHelpers::addClassModifier($this->flags, Stmt\Class_::MODIFIER_FINAL);
 
         return $this;
     }
 
-    /**
-     * Makes the class readonly.
-     *
-     * @return $this The builder instance (for fluid interface)
-     */
     public function makeReadonly() {
-        $this->flags = BuilderHelpers::addClassModifier($this->flags, Modifiers::READONLY);
+        $this->flags = BuilderHelpers::addClassModifier($this->flags, Stmt\Class_::MODIFIER_READONLY);
 
         return $this;
     }
@@ -111,18 +99,20 @@ class Class_ extends Declaration {
     public function addStmt($stmt) {
         $stmt = BuilderHelpers::normalizeNode($stmt);
 
-        if ($stmt instanceof Stmt\Property) {
-            $this->properties[] = $stmt;
-        } elseif ($stmt instanceof Stmt\ClassMethod) {
-            $this->methods[] = $stmt;
-        } elseif ($stmt instanceof Stmt\TraitUse) {
-            $this->uses[] = $stmt;
-        } elseif ($stmt instanceof Stmt\ClassConst) {
-            $this->constants[] = $stmt;
-        } else {
+        $targets = [
+            Stmt\TraitUse::class    => &$this->uses,
+            Stmt\ClassConst::class  => &$this->constants,
+            Stmt\Property::class    => &$this->properties,
+            Stmt\ClassMethod::class => &$this->methods,
+        ];
+
+        $class = \get_class($stmt);
+        if (!isset($targets[$class])) {
             throw new \LogicException(sprintf('Unexpected node of type "%s"', $stmt->getType()));
         }
 
+        $targets[$class][] = $stmt;
+
         return $this;
     }
 
@@ -144,7 +134,7 @@ class Class_ extends Declaration {
      *
      * @return Stmt\Class_ The built class node
      */
-    public function getNode(): PhpParser\Node {
+    public function getNode() : PhpParser\Node {
         return new Stmt\Class_($this->name, [
             'flags' => $this->flags,
             'extends' => $this->extends,

lib/PhpParser/Builder/Declaration.php

@@ -5,23 +5,16 @@ namespace PhpParser\Builder;
 use PhpParser;
 use PhpParser\BuilderHelpers;
 
-abstract class Declaration implements PhpParser\Builder {
-    /** @var array<string, mixed> */
+abstract class Declaration implements PhpParser\Builder
+{
     protected $attributes = [];
 
-    /**
-     * Adds a statement.
-     *
-     * @param PhpParser\Node\Stmt|PhpParser\Builder $stmt The statement to add
-     *
-     * @return $this The builder instance (for fluid interface)
-     */
     abstract public function addStmt($stmt);
 
     /**
      * Adds multiple statements.
      *
-     * @param (PhpParser\Node\Stmt|PhpParser\Builder)[] $stmts The statements to add
+     * @param array $stmts The statements to add
      *
      * @return $this The builder instance (for fluid interface)
      */

lib/PhpParser/Builder/EnumCase.php

@@ -10,15 +10,13 @@ use PhpParser\Node;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Stmt;
 
-class EnumCase implements PhpParser\Builder {
-    /** @var Identifier|string */
+class EnumCase implements PhpParser\Builder
+{
     protected $name;
-    /** @var ?Node\Expr */
     protected $value = null;
-    /** @var array<string, mixed> */
     protected $attributes = [];
 
-    /** @var list<Node\AttributeGroup> */
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**

lib/PhpParser/Builder/Enum_.php

@@ -9,22 +9,19 @@ use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 use PhpParser\Node\Stmt;
 
-class Enum_ extends Declaration {
-    /** @var string */
+class Enum_ extends Declaration
+{
     protected $name;
-    /** @var Identifier|null */
     protected $scalarType = null;
-    /** @var list<Name> */
+
     protected $implements = [];
-    /** @var list<Stmt\TraitUse> */
+
     protected $uses = [];
-    /** @var list<Stmt\EnumCase> */
     protected $enumCases = [];
-    /** @var list<Stmt\ClassConst> */
     protected $constants = [];
-    /** @var list<Stmt\ClassMethod> */
     protected $methods = [];
-    /** @var list<Node\AttributeGroup> */
+
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -39,7 +36,7 @@ class Enum_ extends Declaration {
     /**
      * Sets the scalar type.
      *
-     * @param string|Identifier $scalarType
+     * @param string|Identifier $type
      *
      * @return $this
      */
@@ -74,18 +71,20 @@ class Enum_ extends Declaration {
     public function addStmt($stmt) {
         $stmt = BuilderHelpers::normalizeNode($stmt);
 
-        if ($stmt instanceof Stmt\EnumCase) {
-            $this->enumCases[] = $stmt;
-        } elseif ($stmt instanceof Stmt\ClassMethod) {
-            $this->methods[] = $stmt;
-        } elseif ($stmt instanceof Stmt\TraitUse) {
-            $this->uses[] = $stmt;
-        } elseif ($stmt instanceof Stmt\ClassConst) {
-            $this->constants[] = $stmt;
-        } else {
+        $targets = [
+            Stmt\TraitUse::class    => &$this->uses,
+            Stmt\EnumCase::class    => &$this->enumCases,
+            Stmt\ClassConst::class  => &$this->constants,
+            Stmt\ClassMethod::class => &$this->methods,
+        ];
+
+        $class = \get_class($stmt);
+        if (!isset($targets[$class])) {
             throw new \LogicException(sprintf('Unexpected node of type "%s"', $stmt->getType()));
         }
 
+        $targets[$class][] = $stmt;
+
         return $this;
     }
 
@@ -107,7 +106,7 @@ class Enum_ extends Declaration {
      *
      * @return Stmt\Enum_ The built enum node
      */
-    public function getNode(): PhpParser\Node {
+    public function getNode() : PhpParser\Node {
         return new Stmt\Enum_($this->name, [
             'scalarType' => $this->scalarType,
             'implements' => $this->implements,

lib/PhpParser/Builder/FunctionLike.php

@@ -5,13 +5,12 @@ namespace PhpParser\Builder;
 use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 
-abstract class FunctionLike extends Declaration {
-    /** @var bool */
+abstract class FunctionLike extends Declaration
+{
     protected $returnByRef = false;
-    /** @var Node\Param[] */
     protected $params = [];
 
-    /** @var Node\Identifier|Node\Name|Node\ComplexType|null */
+    /** @var string|Node\Name|Node\NullableType|null */
     protected $returnType = null;
 
     /**
@@ -47,7 +46,7 @@ abstract class FunctionLike extends Declaration {
     /**
      * Adds multiple parameters.
      *
-     * @param (Node\Param|Param)[] $params The parameters to add
+     * @param array $params The parameters to add
      *
      * @return $this The builder instance (for fluid interface)
      */

lib/PhpParser/Builder/Function_.php

@@ -7,13 +7,12 @@ use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class Function_ extends FunctionLike {
-    /** @var string */
+class Function_ extends FunctionLike
+{
     protected $name;
-    /** @var list<Stmt> */
     protected $stmts = [];
 
-    /** @var list<Node\AttributeGroup> */
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -56,7 +55,7 @@ class Function_ extends FunctionLike {
      *
      * @return Stmt\Function_ The built function node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         return new Stmt\Function_($this->name, [
             'byRef'      => $this->returnByRef,
             'params'     => $this->params,

lib/PhpParser/Builder/Interface_.php

@@ -8,16 +8,14 @@ use PhpParser\Node;
 use PhpParser\Node\Name;
 use PhpParser\Node\Stmt;
 
-class Interface_ extends Declaration {
-    /** @var string */
+class Interface_ extends Declaration
+{
     protected $name;
-    /** @var list<Name> */
     protected $extends = [];
-    /** @var list<Stmt\ClassConst> */
     protected $constants = [];
-    /** @var list<Stmt\ClassMethod> */
     protected $methods = [];
-    /** @var list<Node\AttributeGroup> */
+
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -85,7 +83,7 @@ class Interface_ extends Declaration {
      *
      * @return Stmt\Interface_ The built interface node
      */
-    public function getNode(): PhpParser\Node {
+    public function getNode() : PhpParser\Node {
         return new Stmt\Interface_($this->name, [
             'extends' => $this->extends,
             'stmts' => array_merge($this->constants, $this->methods),

lib/PhpParser/Builder/Method.php

@@ -4,20 +4,18 @@ namespace PhpParser\Builder;
 
 use PhpParser;
 use PhpParser\BuilderHelpers;
-use PhpParser\Modifiers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class Method extends FunctionLike {
-    /** @var string */
+class Method extends FunctionLike
+{
     protected $name;
-    /** @var int */
     protected $flags = 0;
 
-    /** @var list<Stmt>|null */
+    /** @var array|null */
     protected $stmts = [];
 
-    /** @var list<Node\AttributeGroup> */
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -35,7 +33,7 @@ class Method extends FunctionLike {
      * @return $this The builder instance (for fluid interface)
      */
     public function makePublic() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PUBLIC);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PUBLIC);
 
         return $this;
     }
@@ -46,7 +44,7 @@ class Method extends FunctionLike {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeProtected() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PROTECTED);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PROTECTED);
 
         return $this;
     }
@@ -57,7 +55,7 @@ class Method extends FunctionLike {
      * @return $this The builder instance (for fluid interface)
      */
     public function makePrivate() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PRIVATE);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PRIVATE);
 
         return $this;
     }
@@ -68,7 +66,7 @@ class Method extends FunctionLike {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeStatic() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::STATIC);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_STATIC);
 
         return $this;
     }
@@ -83,7 +81,7 @@ class Method extends FunctionLike {
             throw new \LogicException('Cannot make method with statements abstract');
         }
 
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::ABSTRACT);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_ABSTRACT);
         $this->stmts = null; // abstract methods don't have statements
 
         return $this;
@@ -95,7 +93,7 @@ class Method extends FunctionLike {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeFinal() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::FINAL);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_FINAL);
 
         return $this;
     }
@@ -135,7 +133,7 @@ class Method extends FunctionLike {
      *
      * @return Stmt\ClassMethod The built method node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         return new Stmt\ClassMethod($this->name, [
             'flags'      => $this->flags,
             'byRef'      => $this->returnByRef,

lib/PhpParser/Builder/Namespace_.php

@@ -7,10 +7,9 @@ use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class Namespace_ extends Declaration {
-    /** @var Node\Name|null */
+class Namespace_ extends Declaration
+{
     private $name;
-    /** @var Stmt[] */
     private $stmts = [];
 
     /**
@@ -40,7 +39,7 @@ class Namespace_ extends Declaration {
      *
      * @return Stmt\Namespace_ The built node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         return new Stmt\Namespace_($this->name, $this->stmts, $this->attributes);
     }
 }

lib/PhpParser/Builder/Param.php

@@ -6,18 +6,22 @@ use PhpParser;
 use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 
-class Param implements PhpParser\Builder {
-    /** @var string */
+class Param implements PhpParser\Builder
+{
     protected $name;
-    /** @var Node\Expr|null */
+
     protected $default = null;
-    /** @var Node\Identifier|Node\Name|Node\ComplexType|null */
+
+    /** @var Node\Identifier|Node\Name|Node\NullableType|null */
     protected $type = null;
-    /** @var bool */
+
     protected $byRef = false;
-    /** @var bool */
+
     protected $variadic = false;
-    /** @var list<Node\AttributeGroup> */
+
+    protected $flags = 0;
+
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -58,6 +62,19 @@ class Param implements PhpParser\Builder {
         return $this;
     }
 
+    /**
+     * Sets type for the parameter.
+     *
+     * @param string|Node\Name|Node\Identifier|Node\ComplexType $type Parameter type
+     *
+     * @return $this The builder instance (for fluid interface)
+     *
+     * @deprecated Use setType() instead
+     */
+    public function setTypeHint($type) {
+        return $this->setType($type);
+    }
+
     /**
      * Make the parameter accept the value by reference.
      *
@@ -80,6 +97,50 @@ class Param implements PhpParser\Builder {
         return $this;
     }
 
+    /**
+     * Makes the (promoted) parameter public.
+     *
+     * @return $this The builder instance (for fluid interface)
+     */
+    public function makePublic() {
+        $this->flags = BuilderHelpers::addModifier($this->flags, Node\Stmt\Class_::MODIFIER_PUBLIC);
+
+        return $this;
+    }
+
+    /**
+     * Makes the (promoted) parameter protected.
+     *
+     * @return $this The builder instance (for fluid interface)
+     */
+    public function makeProtected() {
+        $this->flags = BuilderHelpers::addModifier($this->flags, Node\Stmt\Class_::MODIFIER_PROTECTED);
+
+        return $this;
+    }
+
+    /**
+     * Makes the (promoted) parameter private.
+     *
+     * @return $this The builder instance (for fluid interface)
+     */
+    public function makePrivate() {
+        $this->flags = BuilderHelpers::addModifier($this->flags, Node\Stmt\Class_::MODIFIER_PRIVATE);
+
+        return $this;
+    }
+
+    /**
+     * Makes the (promoted) parameter readonly.
+     *
+     * @return $this The builder instance (for fluid interface)
+     */
+    public function makeReadonly() {
+        $this->flags = BuilderHelpers::addModifier($this->flags, Node\Stmt\Class_::MODIFIER_READONLY);
+
+        return $this;
+    }
+
     /**
      * Adds an attribute group.
      *
@@ -98,10 +159,10 @@ class Param implements PhpParser\Builder {
      *
      * @return Node\Param The built parameter node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         return new Node\Param(
             new Node\Expr\Variable($this->name),
-            $this->default, $this->type, $this->byRef, $this->variadic, [], 0, $this->attributeGroups
+            $this->default, $this->type, $this->byRef, $this->variadic, [], $this->flags, $this->attributeGroups
         );
     }
 }

lib/PhpParser/Builder/Property.php

@@ -4,25 +4,24 @@ namespace PhpParser\Builder;
 
 use PhpParser;
 use PhpParser\BuilderHelpers;
-use PhpParser\Modifiers;
 use PhpParser\Node;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 use PhpParser\Node\Stmt;
 use PhpParser\Node\ComplexType;
 
-class Property implements PhpParser\Builder {
-    /** @var string */
+class Property implements PhpParser\Builder
+{
     protected $name;
-    /** @var int */
+
     protected $flags = 0;
-    /** @var Node\Expr|null */
     protected $default = null;
-    /** @var array<string, mixed> */
     protected $attributes = [];
-    /** @var null|Identifier|Name|ComplexType */
+
+    /** @var null|Identifier|Name|NullableType */
     protected $type;
-    /** @var list<Node\AttributeGroup> */
+
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -40,7 +39,7 @@ class Property implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makePublic() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PUBLIC);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PUBLIC);
 
         return $this;
     }
@@ -51,7 +50,7 @@ class Property implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeProtected() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PROTECTED);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PROTECTED);
 
         return $this;
     }
@@ -62,7 +61,7 @@ class Property implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makePrivate() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::PRIVATE);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_PRIVATE);
 
         return $this;
     }
@@ -73,7 +72,7 @@ class Property implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeStatic() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::STATIC);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_STATIC);
 
         return $this;
     }
@@ -84,7 +83,7 @@ class Property implements PhpParser\Builder {
      * @return $this The builder instance (for fluid interface)
      */
     public function makeReadonly() {
-        $this->flags = BuilderHelpers::addModifier($this->flags, Modifiers::READONLY);
+        $this->flags = BuilderHelpers::addModifier($this->flags, Stmt\Class_::MODIFIER_READONLY);
 
         return $this;
     }
@@ -148,11 +147,11 @@ class Property implements PhpParser\Builder {
      *
      * @return Stmt\Property The built property node
      */
-    public function getNode(): PhpParser\Node {
+    public function getNode() : PhpParser\Node {
         return new Stmt\Property(
-            $this->flags !== 0 ? $this->flags : Modifiers::PUBLIC,
+            $this->flags !== 0 ? $this->flags : Stmt\Class_::MODIFIER_PUBLIC,
             [
-                new Node\PropertyItem($this->name, $this->default)
+                new Stmt\PropertyProperty($this->name, $this->default)
             ],
             $this->attributes,
             $this->type,

lib/PhpParser/Builder/TraitUse.php

@@ -7,10 +7,9 @@ use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class TraitUse implements Builder {
-    /** @var Node\Name[] */
+class TraitUse implements Builder
+{
     protected $traits = [];
-    /** @var Stmt\TraitUseAdaptation[] */
     protected $adaptations = [];
 
     /**
@@ -59,7 +58,7 @@ class TraitUse implements Builder {
      *
      * @return Node The built node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         return new Stmt\TraitUse($this->traits, $this->adaptations);
     }
 }

lib/PhpParser/Builder/TraitUseAdaptation.php

@@ -4,45 +4,43 @@ namespace PhpParser\Builder;
 
 use PhpParser\Builder;
 use PhpParser\BuilderHelpers;
-use PhpParser\Modifiers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class TraitUseAdaptation implements Builder {
-    private const TYPE_UNDEFINED  = 0;
-    private const TYPE_ALIAS      = 1;
-    private const TYPE_PRECEDENCE = 2;
+class TraitUseAdaptation implements Builder
+{
+    const TYPE_UNDEFINED  = 0;
+    const TYPE_ALIAS      = 1;
+    const TYPE_PRECEDENCE = 2;
 
     /** @var int Type of building adaptation */
     protected $type;
-    /** @var Node\Name|null */
+
     protected $trait;
-    /** @var Node\Identifier */
     protected $method;
-    /** @var int|null */
+
     protected $modifier = null;
-    /** @var Node\Identifier|null */
     protected $alias = null;
-    /** @var Node\Name[] */
+
     protected $insteadof = [];
 
     /**
      * Creates a trait use adaptation builder.
      *
-     * @param Node\Name|string|null  $trait  Name of adapted trait
-     * @param Node\Identifier|string $method Name of adapted method
+     * @param Node\Name|string|null  $trait  Name of adaptated trait
+     * @param Node\Identifier|string $method Name of adaptated method
      */
     public function __construct($trait, $method) {
         $this->type = self::TYPE_UNDEFINED;
 
-        $this->trait = is_null($trait) ? null : BuilderHelpers::normalizeName($trait);
+        $this->trait = is_null($trait)? null: BuilderHelpers::normalizeName($trait);
         $this->method = BuilderHelpers::normalizeIdentifier($method);
     }
 
     /**
      * Sets alias of method.
      *
-     * @param Node\Identifier|string $alias Alias for adapted method
+     * @param Node\Identifier|string $alias Alias for adaptated method
      *
      * @return $this The builder instance (for fluid interface)
      */
@@ -55,37 +53,37 @@ class TraitUseAdaptation implements Builder {
             throw new \LogicException('Cannot set alias for not alias adaptation buider');
         }
 
-        $this->alias = BuilderHelpers::normalizeIdentifier($alias);
+        $this->alias = $alias;
         return $this;
     }
 
     /**
-     * Sets adapted method public.
+     * Sets adaptated method public.
      *
      * @return $this The builder instance (for fluid interface)
      */
     public function makePublic() {
-        $this->setModifier(Modifiers::PUBLIC);
+        $this->setModifier(Stmt\Class_::MODIFIER_PUBLIC);
         return $this;
     }
 
     /**
-     * Sets adapted method protected.
+     * Sets adaptated method protected.
      *
      * @return $this The builder instance (for fluid interface)
      */
     public function makeProtected() {
-        $this->setModifier(Modifiers::PROTECTED);
+        $this->setModifier(Stmt\Class_::MODIFIER_PROTECTED);
         return $this;
     }
 
     /**
-     * Sets adapted method private.
+     * Sets adaptated method private.
      *
      * @return $this The builder instance (for fluid interface)
      */
     public function makePrivate() {
-        $this->setModifier(Modifiers::PRIVATE);
+        $this->setModifier(Stmt\Class_::MODIFIER_PRIVATE);
         return $this;
     }
 
@@ -116,7 +114,7 @@ class TraitUseAdaptation implements Builder {
         return $this;
     }
 
-    protected function setModifier(int $modifier): void {
+    protected function setModifier(int $modifier) {
         if ($this->type === self::TYPE_UNDEFINED) {
             $this->type = self::TYPE_ALIAS;
         }
@@ -137,7 +135,7 @@ class TraitUseAdaptation implements Builder {
      *
      * @return Node The built node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         switch ($this->type) {
             case self::TYPE_ALIAS:
                 return new Stmt\TraitUseAdaptation\Alias($this->trait, $this->method, $this->modifier, $this->alias);

lib/PhpParser/Builder/Trait_.php

@@ -7,18 +7,14 @@ use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class Trait_ extends Declaration {
-    /** @var string */
+class Trait_ extends Declaration
+{
     protected $name;
-    /** @var list<Stmt\TraitUse> */
     protected $uses = [];
-    /** @var list<Stmt\ClassConst> */
-    protected $constants = [];
-    /** @var list<Stmt\Property> */
     protected $properties = [];
-    /** @var list<Stmt\ClassMethod> */
     protected $methods = [];
-    /** @var list<Node\AttributeGroup> */
+
+    /** @var Node\AttributeGroup[] */
     protected $attributeGroups = [];
 
     /**
@@ -46,8 +42,6 @@ class Trait_ extends Declaration {
             $this->methods[] = $stmt;
         } elseif ($stmt instanceof Stmt\TraitUse) {
             $this->uses[] = $stmt;
-        } elseif ($stmt instanceof Stmt\ClassConst) {
-            $this->constants[] = $stmt;
         } else {
             throw new \LogicException(sprintf('Unexpected node of type "%s"', $stmt->getType()));
         }
@@ -73,10 +67,10 @@ class Trait_ extends Declaration {
      *
      * @return Stmt\Trait_ The built interface node
      */
-    public function getNode(): PhpParser\Node {
+    public function getNode() : PhpParser\Node {
         return new Stmt\Trait_(
             $this->name, [
-                'stmts' => array_merge($this->uses, $this->constants, $this->properties, $this->methods),
+                'stmts' => array_merge($this->uses, $this->properties, $this->methods),
                 'attrGroups' => $this->attributeGroups,
             ], $this->attributes
         );

lib/PhpParser/Builder/Use_.php

@@ -7,12 +7,10 @@ use PhpParser\BuilderHelpers;
 use PhpParser\Node;
 use PhpParser\Node\Stmt;
 
-class Use_ implements Builder {
-    /** @var Node\Name */
+class Use_ implements Builder
+{
     protected $name;
-    /** @var int */
     protected $type;
-    /** @var string|null */
     protected $alias = null;
 
     /**
@@ -43,9 +41,9 @@ class Use_ implements Builder {
      *
      * @return Stmt\Use_ The built node
      */
-    public function getNode(): Node {
+    public function getNode() : Node {
         return new Stmt\Use_([
-            new Node\UseItem($this->name, $this->alias)
+            new Stmt\UseUse($this->name, $this->alias)
         ], $this->type);
     }
 }

lib/PhpParser/BuilderFactory.php

@@ -10,7 +10,8 @@ use PhpParser\Node\Name;
 use PhpParser\Node\Scalar\String_;
 use PhpParser\Node\Stmt\Use_;
 
-class BuilderFactory {
+class BuilderFactory
+{
     /**
      * Creates an attribute node.
      *
@@ -19,7 +20,7 @@ class BuilderFactory {
      *
      * @return Node\Attribute
      */
-    public function attribute($name, array $args = []): Node\Attribute {
+    public function attribute($name, array $args = []) : Node\Attribute {
         return new Node\Attribute(
             BuilderHelpers::normalizeName($name),
             $this->args($args)
@@ -33,7 +34,7 @@ class BuilderFactory {
      *
      * @return Builder\Namespace_ The created namespace builder
      */
-    public function namespace($name): Builder\Namespace_ {
+    public function namespace($name) : Builder\Namespace_ {
         return new Builder\Namespace_($name);
     }
 
@@ -44,7 +45,7 @@ class BuilderFactory {
      *
      * @return Builder\Class_ The created class builder
      */
-    public function class(string $name): Builder\Class_ {
+    public function class(string $name) : Builder\Class_ {
         return new Builder\Class_($name);
     }
 
@@ -55,7 +56,7 @@ class BuilderFactory {
      *
      * @return Builder\Interface_ The created interface builder
      */
-    public function interface(string $name): Builder\Interface_ {
+    public function interface(string $name) : Builder\Interface_ {
         return new Builder\Interface_($name);
     }
 
@@ -66,7 +67,7 @@ class BuilderFactory {
      *
      * @return Builder\Trait_ The created trait builder
      */
-    public function trait(string $name): Builder\Trait_ {
+    public function trait(string $name) : Builder\Trait_ {
         return new Builder\Trait_($name);
     }
 
@@ -77,7 +78,7 @@ class BuilderFactory {
      *
      * @return Builder\Enum_ The created enum builder
      */
-    public function enum(string $name): Builder\Enum_ {
+    public function enum(string $name) : Builder\Enum_ {
         return new Builder\Enum_($name);
     }
 
@@ -86,9 +87,9 @@ class BuilderFactory {
      *
      * @param Node\Name|string ...$traits Trait names
      *
-     * @return Builder\TraitUse The created trait use builder
+     * @return Builder\TraitUse The create trait use builder
      */
-    public function useTrait(...$traits): Builder\TraitUse {
+    public function useTrait(...$traits) : Builder\TraitUse {
         return new Builder\TraitUse(...$traits);
     }
 
@@ -98,9 +99,9 @@ class BuilderFactory {
      * @param Node\Name|string|null  $trait  Trait name
      * @param Node\Identifier|string $method Method name
      *
-     * @return Builder\TraitUseAdaptation The created trait use adaptation builder
+     * @return Builder\TraitUseAdaptation The create trait use adaptation builder
      */
-    public function traitUseAdaptation($trait, $method = null): Builder\TraitUseAdaptation {
+    public function traitUseAdaptation($trait, $method = null) : Builder\TraitUseAdaptation {
         if ($method === null) {
             $method = $trait;
             $trait = null;
@@ -116,7 +117,7 @@ class BuilderFactory {
      *
      * @return Builder\Method The created method builder
      */
-    public function method(string $name): Builder\Method {
+    public function method(string $name) : Builder\Method {
         return new Builder\Method($name);
     }
 
@@ -127,7 +128,7 @@ class BuilderFactory {
      *
      * @return Builder\Param The created parameter builder
      */
-    public function param(string $name): Builder\Param {
+    public function param(string $name) : Builder\Param {
         return new Builder\Param($name);
     }
 
@@ -138,7 +139,7 @@ class BuilderFactory {
      *
      * @return Builder\Property The created property builder
      */
-    public function property(string $name): Builder\Property {
+    public function property(string $name) : Builder\Property {
         return new Builder\Property($name);
     }
 
@@ -149,7 +150,7 @@ class BuilderFactory {
      *
      * @return Builder\Function_ The created function builder
      */
-    public function function(string $name): Builder\Function_ {
+    public function function(string $name) : Builder\Function_ {
         return new Builder\Function_($name);
     }
 
@@ -160,7 +161,7 @@ class BuilderFactory {
      *
      * @return Builder\Use_ The created use builder
      */
-    public function use($name): Builder\Use_ {
+    public function use($name) : Builder\Use_ {
         return new Builder\Use_($name, Use_::TYPE_NORMAL);
     }
 
@@ -171,7 +172,7 @@ class BuilderFactory {
      *
      * @return Builder\Use_ The created use function builder
      */
-    public function useFunction($name): Builder\Use_ {
+    public function useFunction($name) : Builder\Use_ {
         return new Builder\Use_($name, Use_::TYPE_FUNCTION);
     }
 
@@ -182,7 +183,7 @@ class BuilderFactory {
      *
      * @return Builder\Use_ The created use const builder
      */
-    public function useConst($name): Builder\Use_ {
+    public function useConst($name) : Builder\Use_ {
         return new Builder\Use_($name, Use_::TYPE_CONSTANT);
     }
 
@@ -194,7 +195,7 @@ class BuilderFactory {
      *
      * @return Builder\ClassConst The created use const builder
      */
-    public function classConst($name, $value): Builder\ClassConst {
+    public function classConst($name, $value) : Builder\ClassConst {
         return new Builder\ClassConst($name, $value);
     }
 
@@ -205,7 +206,7 @@ class BuilderFactory {
      *
      * @return Builder\EnumCase The created use const builder
      */
-    public function enumCase($name): Builder\EnumCase {
+    public function enumCase($name) : Builder\EnumCase {
         return new Builder\EnumCase($name);
     }
 
@@ -216,7 +217,7 @@ class BuilderFactory {
      *
      * @return Expr
      */
-    public function val($value): Expr {
+    public function val($value) : Expr {
         return BuilderHelpers::normalizeValue($value);
     }
 
@@ -227,7 +228,7 @@ class BuilderFactory {
      *
      * @return Expr\Variable
      */
-    public function var($name): Expr\Variable {
+    public function var($name) : Expr\Variable {
         if (!\is_string($name) && !$name instanceof Expr) {
             throw new \LogicException('Variable name must be string or Expr');
         }
@@ -242,9 +243,9 @@ class BuilderFactory {
      *
      * @param array $args List of arguments to normalize
      *
-     * @return list<Arg>
+     * @return Arg[]
      */
-    public function args(array $args): array {
+    public function args(array $args) : array {
         $normalizedArgs = [];
         foreach ($args as $key => $arg) {
             if (!($arg instanceof Arg)) {
@@ -266,7 +267,7 @@ class BuilderFactory {
      *
      * @return Expr\FuncCall
      */
-    public function funcCall($name, array $args = []): Expr\FuncCall {
+    public function funcCall($name, array $args = []) : Expr\FuncCall {
         return new Expr\FuncCall(
             BuilderHelpers::normalizeNameOrExpr($name),
             $this->args($args)
@@ -282,7 +283,7 @@ class BuilderFactory {
      *
      * @return Expr\MethodCall
      */
-    public function methodCall(Expr $var, $name, array $args = []): Expr\MethodCall {
+    public function methodCall(Expr $var, $name, array $args = []) : Expr\MethodCall {
         return new Expr\MethodCall(
             $var,
             BuilderHelpers::normalizeIdentifierOrExpr($name),
@@ -299,7 +300,7 @@ class BuilderFactory {
      *
      * @return Expr\StaticCall
      */
-    public function staticCall($class, $name, array $args = []): Expr\StaticCall {
+    public function staticCall($class, $name, array $args = []) : Expr\StaticCall {
         return new Expr\StaticCall(
             BuilderHelpers::normalizeNameOrExpr($class),
             BuilderHelpers::normalizeIdentifierOrExpr($name),
@@ -315,7 +316,7 @@ class BuilderFactory {
      *
      * @return Expr\New_
      */
-    public function new($class, array $args = []): Expr\New_ {
+    public function new($class, array $args = []) : Expr\New_ {
         return new Expr\New_(
             BuilderHelpers::normalizeNameOrExpr($class),
             $this->args($args)
@@ -329,7 +330,7 @@ class BuilderFactory {
      *
      * @return Expr\ConstFetch
      */
-    public function constFetch($name): Expr\ConstFetch {
+    public function constFetch($name) : Expr\ConstFetch {
         return new Expr\ConstFetch(BuilderHelpers::normalizeName($name));
     }
 
@@ -341,7 +342,7 @@ class BuilderFactory {
      *
      * @return Expr\PropertyFetch
      */
-    public function propertyFetch(Expr $var, $name): Expr\PropertyFetch {
+    public function propertyFetch(Expr $var, $name) : Expr\PropertyFetch {
         return new Expr\PropertyFetch($var, BuilderHelpers::normalizeIdentifierOrExpr($name));
     }
 
@@ -367,7 +368,7 @@ class BuilderFactory {
      *
      * @return Concat
      */
-    public function concat(...$exprs): Concat {
+    public function concat(...$exprs) : Concat {
         $numExprs = count($exprs);
         if ($numExprs < 2) {
             throw new \LogicException('Expected at least two expressions');
@@ -384,7 +385,7 @@ class BuilderFactory {
      * @param string|Expr $expr
      * @return Expr
      */
-    private function normalizeStringExpr($expr): Expr {
+    private function normalizeStringExpr($expr) : Expr {
         if ($expr instanceof Expr) {
             return $expr;
         }

lib/PhpParser/BuilderHelpers.php

@@ -15,7 +15,8 @@ use PhpParser\Node\Stmt;
  *
  * @internal
  */
-final class BuilderHelpers {
+final class BuilderHelpers
+{
     /**
      * Normalizes a node: Converts builder objects to nodes.
      *
@@ -23,7 +24,7 @@ final class BuilderHelpers {
      *
      * @return Node The normalized node
      */
-    public static function normalizeNode($node): Node {
+    public static function normalizeNode($node) : Node {
         if ($node instanceof Builder) {
             return $node->getNode();
         }
@@ -44,7 +45,7 @@ final class BuilderHelpers {
      *
      * @return Stmt The normalized statement node
      */
-    public static function normalizeStmt($node): Stmt {
+    public static function normalizeStmt($node) : Stmt {
         $node = self::normalizeNode($node);
         if ($node instanceof Stmt) {
             return $node;
@@ -64,7 +65,7 @@ final class BuilderHelpers {
      *
      * @return Identifier The normalized identifier
      */
-    public static function normalizeIdentifier($name): Identifier {
+    public static function normalizeIdentifier($name) : Identifier {
         if ($name instanceof Identifier) {
             return $name;
         }
@@ -102,7 +103,7 @@ final class BuilderHelpers {
      *
      * @return Name The normalized name
      */
-    public static function normalizeName($name): Name {
+    public static function normalizeName($name) : Name {
         if ($name instanceof Name) {
             return $name;
         }
@@ -218,7 +219,7 @@ final class BuilderHelpers {
      *
      * @return Expr The normalized value
      */
-    public static function normalizeValue($value): Expr {
+    public static function normalizeValue($value) : Expr {
         if ($value instanceof Node\Expr) {
             return $value;
         }
@@ -236,11 +237,11 @@ final class BuilderHelpers {
         }
 
         if (is_int($value)) {
-            return new Scalar\Int_($value);
+            return new Scalar\LNumber($value);
         }
 
         if (is_float($value)) {
-            return new Scalar\Float_($value);
+            return new Scalar\DNumber($value);
         }
 
         if (is_string($value)) {
@@ -253,12 +254,12 @@ final class BuilderHelpers {
             foreach ($value as $itemKey => $itemValue) {
                 // for consecutive, numeric keys don't generate keys
                 if (null !== $lastKey && ++$lastKey === $itemKey) {
-                    $items[] = new Node\ArrayItem(
+                    $items[] = new Expr\ArrayItem(
                         self::normalizeValue($itemValue)
                     );
                 } else {
                     $lastKey = null;
-                    $items[] = new Node\ArrayItem(
+                    $items[] = new Expr\ArrayItem(
                         self::normalizeValue($itemValue),
                         self::normalizeValue($itemKey)
                     );
@@ -278,7 +279,7 @@ final class BuilderHelpers {
      *
      * @return Comment\Doc The normalized doc comment
      */
-    public static function normalizeDocComment($docComment): Comment\Doc {
+    public static function normalizeDocComment($docComment) : Comment\Doc {
         if ($docComment instanceof Comment\Doc) {
             return $docComment;
         }
@@ -297,7 +298,8 @@ final class BuilderHelpers {
      *
      * @return Node\AttributeGroup The Attribute Group
      */
-    public static function normalizeAttribute($attribute): Node\AttributeGroup {
+    public static function normalizeAttribute($attribute) : Node\AttributeGroup
+    {
         if ($attribute instanceof Node\AttributeGroup) {
             return $attribute;
         }
@@ -317,8 +319,8 @@ final class BuilderHelpers {
      *
      * @return int New modifiers
      */
-    public static function addModifier(int $modifiers, int $modifier): int {
-        Modifiers::verifyModifier($modifiers, $modifier);
+    public static function addModifier(int $modifiers, int $modifier) : int {
+        Stmt\Class_::verifyModifier($modifiers, $modifier);
         return $modifiers | $modifier;
     }
 
@@ -326,8 +328,8 @@ final class BuilderHelpers {
      * Adds a modifier and returns new modifier bitmask.
      * @return int New modifiers
      */
-    public static function addClassModifier(int $existingModifiers, int $modifierToSet): int {
-        Modifiers::verifyClassModifier($existingModifiers, $modifierToSet);
+    public static function addClassModifier(int $existingModifiers, int $modifierToSet) : int {
+        Stmt\Class_::verifyClassModifier($existingModifiers, $modifierToSet);
         return $existingModifiers | $modifierToSet;
     }
 }

lib/PhpParser/Comment.php

@@ -2,20 +2,14 @@
 
 namespace PhpParser;
 
-class Comment implements \JsonSerializable {
-    /** @var string string */
+class Comment implements \JsonSerializable
+{
     protected $text;
-    /** @var int */
     protected $startLine;
-    /** @var int */
     protected $startFilePos;
-    /** @var int */
     protected $startTokenPos;
-    /** @var int */
     protected $endLine;
-    /** @var int */
     protected $endFilePos;
-    /** @var int */
     protected $endTokenPos;
 
     /**
@@ -45,7 +39,7 @@ class Comment implements \JsonSerializable {
      *
      * @return string The comment text (including comment delimiters like /*)
      */
-    public function getText(): string {
+    public function getText() : string {
         return $this->text;
     }
 
@@ -54,7 +48,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int Line number (or -1 if not available)
      */
-    public function getStartLine(): int {
+    public function getStartLine() : int {
         return $this->startLine;
     }
 
@@ -63,7 +57,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int File offset (or -1 if not available)
      */
-    public function getStartFilePos(): int {
+    public function getStartFilePos() : int {
         return $this->startFilePos;
     }
 
@@ -72,7 +66,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int Token offset (or -1 if not available)
      */
-    public function getStartTokenPos(): int {
+    public function getStartTokenPos() : int {
         return $this->startTokenPos;
     }
 
@@ -81,7 +75,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int Line number (or -1 if not available)
      */
-    public function getEndLine(): int {
+    public function getEndLine() : int {
         return $this->endLine;
     }
 
@@ -90,7 +84,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int File offset (or -1 if not available)
      */
-    public function getEndFilePos(): int {
+    public function getEndFilePos() : int {
         return $this->endFilePos;
     }
 
@@ -99,7 +93,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int Token offset (or -1 if not available)
      */
-    public function getEndTokenPos(): int {
+    public function getEndTokenPos() : int {
         return $this->endTokenPos;
     }
 
@@ -110,7 +104,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int Line number
      */
-    public function getLine(): int {
+    public function getLine() : int {
         return $this->startLine;
     }
 
@@ -121,7 +115,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int File offset
      */
-    public function getFilePos(): int {
+    public function getFilePos() : int {
         return $this->startFilePos;
     }
 
@@ -132,7 +126,7 @@ class Comment implements \JsonSerializable {
      *
      * @return int Token offset
      */
-    public function getTokenPos(): int {
+    public function getTokenPos() : int {
         return $this->startTokenPos;
     }
 
@@ -141,7 +135,7 @@ class Comment implements \JsonSerializable {
      *
      * @return string The comment text (including comment delimiters like /*)
      */
-    public function __toString(): string {
+    public function __toString() : string {
         return $this->text;
     }
 
@@ -156,13 +150,12 @@ class Comment implements \JsonSerializable {
      * @return mixed|string
      */
     public function getReformattedText() {
-        $text = $this->text;
+        $text = trim($this->text);
         $newlinePos = strpos($text, "\n");
         if (false === $newlinePos) {
             // Single line comments don't need further processing
             return $text;
-        }
-        if (preg_match('((*BSR_ANYCRLF)(*ANYCRLF)^.*(?:\R\s+\*.*)+$)', $text)) {
+        } elseif (preg_match('((*BSR_ANYCRLF)(*ANYCRLF)^.*(?:\R\s+\*.*)+$)', $text)) {
             // Multi line comment of the type
             //
             //     /*
@@ -172,8 +165,7 @@ class Comment implements \JsonSerializable {
             //
             // is handled by replacing the whitespace sequences before the * by a single space
             return preg_replace('(^\s+\*)m', ' *', $this->text);
-        }
-        if (preg_match('(^/\*\*?\s*[\r\n])', $text) && preg_match('(\n(\s*)\*/$)', $text, $matches)) {
+        } elseif (preg_match('(^/\*\*?\s*[\r\n])', $text) && preg_match('(\n(\s*)\*/$)', $text, $matches)) {
             // Multi line comment of the type
             //
             //    /*
@@ -185,8 +177,7 @@ class Comment implements \JsonSerializable {
             // */ on all lines. So if the last line is "    */", then "    " is removed at the
             // start of all lines.
             return preg_replace('(^' . preg_quote($matches[1]) . ')m', '', $text);
-        }
-        if (preg_match('(^/\*\*?\s*(?!\s))', $text, $matches)) {
+        } elseif (preg_match('(^/\*\*?\s*(?!\s))', $text, $matches)) {
             // Multi line comment of the type
             //
             //     /* Some text.
@@ -213,9 +204,9 @@ class Comment implements \JsonSerializable {
      * @param string $str String to check
      * @return int Length in characters. Tabs count as single characters.
      */
-    private function getShortestWhitespacePrefixLen(string $str): int {
+    private function getShortestWhitespacePrefixLen(string $str) : int {
         $lines = explode("\n", $str);
-        $shortestPrefixLen = \PHP_INT_MAX;
+        $shortestPrefixLen = \INF;
         foreach ($lines as $line) {
             preg_match('(^\s*)', $line, $matches);
             $prefixLen = strlen($matches[0]);
@@ -230,7 +221,7 @@ class Comment implements \JsonSerializable {
      * @return       array
      * @psalm-return array{nodeType:string, text:mixed, line:mixed, filePos:mixed}
      */
-    public function jsonSerialize(): array {
+    public function jsonSerialize() : array {
         // Technically not a node, but we make it look like one anyway
         $type = $this instanceof Comment\Doc ? 'Comment_Doc' : 'Comment';
         return [

lib/PhpParser/Comment/Doc.php

@@ -2,5 +2,6 @@
 
 namespace PhpParser\Comment;
 
-class Doc extends \PhpParser\Comment {
+class Doc extends \PhpParser\Comment
+{
 }

lib/PhpParser/ConstExprEvaluationException.php

@@ -1,6 +1,6 @@
-<?php declare(strict_types=1);
+<?php
 
 namespace PhpParser;
 
-class ConstExprEvaluationException extends \Exception {
-}
+class ConstExprEvaluationException extends \Exception
+{}

lib/PhpParser/ConstExprEvaluator.php

@@ -1,12 +1,11 @@
-<?php declare(strict_types=1);
+<?php
 
 namespace PhpParser;
 
+use function array_merge;
 use PhpParser\Node\Expr;
 use PhpParser\Node\Scalar;
 
-use function array_merge;
-
 /**
  * Evaluates constant expressions.
  *
@@ -26,8 +25,8 @@ use function array_merge;
  * point to string conversions are affected by the precision ini setting. Secondly, they are also
  * affected by the LC_NUMERIC locale.
  */
-class ConstExprEvaluator {
-    /** @var callable|null */
+class ConstExprEvaluator
+{
     private $fallbackEvaluator;
 
     /**
@@ -38,8 +37,8 @@ class ConstExprEvaluator {
      *
      * @param callable|null $fallbackEvaluator To call if subexpression cannot be evaluated
      */
-    public function __construct(?callable $fallbackEvaluator = null) {
-        $this->fallbackEvaluator = $fallbackEvaluator ?? function (Expr $expr) {
+    public function __construct(callable $fallbackEvaluator = null) {
+        $this->fallbackEvaluator = $fallbackEvaluator ?? function(Expr $expr) {
             throw new ConstExprEvaluationException(
                 "Expression of type {$expr->getType()} cannot be evaluated"
             );
@@ -64,7 +63,7 @@ class ConstExprEvaluator {
      * @throws ConstExprEvaluationException if the expression cannot be evaluated or an error occurred
      */
     public function evaluateSilently(Expr $expr) {
-        set_error_handler(function ($num, $str, $file, $line) {
+        set_error_handler(function($num, $str, $file, $line) {
             throw new \ErrorException($str, 0, $num, $file, $line);
         });
 
@@ -102,10 +101,9 @@ class ConstExprEvaluator {
         return $this->evaluate($expr);
     }
 
-    /** @return mixed */
     private function evaluate(Expr $expr) {
-        if ($expr instanceof Scalar\Int_
-            || $expr instanceof Scalar\Float_
+        if ($expr instanceof Scalar\LNumber
+            || $expr instanceof Scalar\DNumber
             || $expr instanceof Scalar\String_
         ) {
             return $expr->value;
@@ -148,7 +146,7 @@ class ConstExprEvaluator {
         return ($this->fallbackEvaluator)($expr);
     }
 
-    private function evaluateArray(Expr\Array_ $expr): array {
+    private function evaluateArray(Expr\Array_ $expr) {
         $array = [];
         foreach ($expr->items as $item) {
             if (null !== $item->key) {
@@ -162,7 +160,6 @@ class ConstExprEvaluator {
         return $array;
     }
 
-    /** @return mixed */
     private function evaluateTernary(Expr\Ternary $expr) {
         if (null === $expr->if) {
             return $this->evaluate($expr->cond) ?: $this->evaluate($expr->else);
@@ -173,7 +170,6 @@ class ConstExprEvaluator {
             : $this->evaluate($expr->else);
     }
 
-    /** @return mixed */
     private function evaluateBinaryOp(Expr\BinaryOp $expr) {
         if ($expr instanceof Expr\BinaryOp\Coalesce
             && $expr->left instanceof Expr\ArrayDimFetch
@@ -220,7 +216,6 @@ class ConstExprEvaluator {
         throw new \Exception('Should not happen');
     }
 
-    /** @return mixed */
     private function evaluateConstFetch(Expr\ConstFetch $expr) {
         $name = $expr->name->toLowerString();
         switch ($name) {

lib/PhpParser/Error.php

@@ -2,21 +2,25 @@
 
 namespace PhpParser;
 
-class Error extends \RuntimeException {
-    /** @var string */
+class Error extends \RuntimeException
+{
     protected $rawMessage;
-    /** @var array<string, mixed> */
     protected $attributes;
 
     /**
      * Creates an Exception signifying a parse error.
      *
-     * @param string $message Error message
-     * @param array<string, mixed> $attributes Attributes of node/token where error occurred
+     * @param string    $message    Error message
+     * @param array|int $attributes Attributes of node/token where error occurred
+     *                              (or start line of error -- deprecated)
      */
-    public function __construct(string $message, array $attributes = []) {
+    public function __construct(string $message, $attributes = []) {
         $this->rawMessage = $message;
-        $this->attributes = $attributes;
+        if (is_array($attributes)) {
+            $this->attributes = $attributes;
+        } else {
+            $this->attributes = ['startLine' => $attributes];
+        }
         $this->updateMessage();
     }
 
@@ -25,7 +29,7 @@ class Error extends \RuntimeException {
      *
      * @return string Error message
      */
-    public function getRawMessage(): string {
+    public function getRawMessage() : string {
         return $this->rawMessage;
     }
 
@@ -34,7 +38,7 @@ class Error extends \RuntimeException {
      *
      * @return int Error start line
      */
-    public function getStartLine(): int {
+    public function getStartLine() : int {
         return $this->attributes['startLine'] ?? -1;
     }
 
@@ -43,25 +47,25 @@ class Error extends \RuntimeException {
      *
      * @return int Error end line
      */
-    public function getEndLine(): int {
+    public function getEndLine() : int {
         return $this->attributes['endLine'] ?? -1;
     }
 
     /**
      * Gets the attributes of the node/token the error occurred at.
      *
-     * @return array<string, mixed>
+     * @return array
      */
-    public function getAttributes(): array {
+    public function getAttributes() : array {
         return $this->attributes;
     }
 
     /**
      * Sets the attributes of the node/token the error occurred at.
      *
-     * @param array<string, mixed> $attributes
+     * @param array $attributes
      */
-    public function setAttributes(array $attributes): void {
+    public function setAttributes(array $attributes) {
         $this->attributes = $attributes;
         $this->updateMessage();
     }
@@ -71,7 +75,7 @@ class Error extends \RuntimeException {
      *
      * @param string $message Error message
      */
-    public function setRawMessage(string $message): void {
+    public function setRawMessage(string $message) {
         $this->rawMessage = $message;
         $this->updateMessage();
     }
@@ -81,7 +85,7 @@ class Error extends \RuntimeException {
      *
      * @param int $line Error start line
      */
-    public function setStartLine(int $line): void {
+    public function setStartLine(int $line) {
         $this->attributes['startLine'] = $line;
         $this->updateMessage();
     }
@@ -93,7 +97,7 @@ class Error extends \RuntimeException {
      *
      * @return bool
      */
-    public function hasColumnInfo(): bool {
+    public function hasColumnInfo() : bool {
         return isset($this->attributes['startFilePos'], $this->attributes['endFilePos']);
     }
 
@@ -103,7 +107,7 @@ class Error extends \RuntimeException {
      * @param string $code Source code of the file
      * @return int
      */
-    public function getStartColumn(string $code): int {
+    public function getStartColumn(string $code) : int {
         if (!$this->hasColumnInfo()) {
             throw new \RuntimeException('Error does not have column information');
         }
@@ -117,7 +121,7 @@ class Error extends \RuntimeException {
      * @param string $code Source code of the file
      * @return int
      */
-    public function getEndColumn(string $code): int {
+    public function getEndColumn(string $code) : int {
         if (!$this->hasColumnInfo()) {
             throw new \RuntimeException('Error does not have column information');
         }
@@ -132,7 +136,7 @@ class Error extends \RuntimeException {
      *
      * @return string Formatted message
      */
-    public function getMessageWithColumnInfo(string $code): string {
+    public function getMessageWithColumnInfo(string $code) : string {
         return sprintf(
             '%s from %d:%d to %d:%d', $this->getRawMessage(),
             $this->getStartLine(), $this->getStartColumn($code),
@@ -148,7 +152,7 @@ class Error extends \RuntimeException {
      *
      * @return int 1-based column (relative to start of line)
      */
-    private function toColumn(string $code, int $pos): int {
+    private function toColumn(string $code, int $pos) : int {
         if ($pos > strlen($code)) {
             throw new \RuntimeException('Invalid position information');
         }
@@ -164,7 +168,7 @@ class Error extends \RuntimeException {
     /**
      * Updates the exception message after a change to rawMessage or rawLine.
      */
-    protected function updateMessage(): void {
+    protected function updateMessage() {
         $this->message = $this->rawMessage;
 
         if (-1 === $this->getStartLine()) {

lib/PhpParser/ErrorHandler.php

@@ -2,11 +2,12 @@
 
 namespace PhpParser;
 
-interface ErrorHandler {
+interface ErrorHandler
+{
     /**
      * Handle an error generated during lexing, parsing or some other operation.
      *
      * @param Error $error The error that needs to be handled
      */
-    public function handleError(Error $error): void;
+    public function handleError(Error $error);
 }

lib/PhpParser/ErrorHandler/Collecting.php

@@ -10,11 +10,12 @@ use PhpParser\ErrorHandler;
  *
  * This allows graceful handling of errors.
  */
-class Collecting implements ErrorHandler {
+class Collecting implements ErrorHandler
+{
     /** @var Error[] Collected errors */
     private $errors = [];
 
-    public function handleError(Error $error): void {
+    public function handleError(Error $error) {
         $this->errors[] = $error;
     }
 
@@ -23,7 +24,7 @@ class Collecting implements ErrorHandler {
      *
      * @return Error[]
      */
-    public function getErrors(): array {
+    public function getErrors() : array {
         return $this->errors;
     }
 
@@ -32,14 +33,14 @@ class Collecting implements ErrorHandler {
      *
      * @return bool
      */
-    public function hasErrors(): bool {
+    public function hasErrors() : bool {
         return !empty($this->errors);
     }
 
     /**
      * Reset/clear collected errors.
      */
-    public function clearErrors(): void {
+    public function clearErrors() {
         $this->errors = [];
     }
 }

lib/PhpParser/ErrorHandler/Throwing.php

@@ -10,8 +10,9 @@ use PhpParser\ErrorHandler;
  *
  * This is the default strategy used by all components.
  */
-class Throwing implements ErrorHandler {
-    public function handleError(Error $error): void {
+class Throwing implements ErrorHandler
+{
+    public function handleError(Error $error) {
         throw $error;
     }
 }

lib/PhpParser/Internal/DiffElem.php

@@ -5,11 +5,12 @@ namespace PhpParser\Internal;
 /**
  * @internal
  */
-class DiffElem {
-    public const TYPE_KEEP = 0;
-    public const TYPE_REMOVE = 1;
-    public const TYPE_ADD = 2;
-    public const TYPE_REPLACE = 3;
+class DiffElem
+{
+    const TYPE_KEEP = 0;
+    const TYPE_REMOVE = 1;
+    const TYPE_ADD = 2;
+    const TYPE_REPLACE = 3;
 
     /** @var int One of the TYPE_* constants */
     public $type;
@@ -18,11 +19,6 @@ class DiffElem {
     /** @var mixed Is null for remove operations */
     public $new;
 
-    /**
-     * @param int $type One of the TYPE_* constants
-     * @param mixed $old Is null for add operations
-     * @param mixed $new Is null for remove operations
-     */
     public function __construct(int $type, $old, $new) {
         $this->type = $type;
         $this->old = $old;

lib/PhpParser/Internal/Differ.php

@@ -8,17 +8,16 @@ namespace PhpParser\Internal;
  * Myers, Eugene W. "An O (ND) difference algorithm and its variations."
  * Algorithmica 1.1 (1986): 251-266.
  *
- * @template T
  * @internal
  */
-class Differ {
-    /** @var callable(T, T): bool */
+class Differ
+{
     private $isEqual;
 
     /**
      * Create differ over the given equality relation.
      *
-     * @param callable(T, T): bool $isEqual Equality relation
+     * @param callable $isEqual Equality relation with signature function($a, $b) : bool
      */
     public function __construct(callable $isEqual) {
         $this->isEqual = $isEqual;
@@ -27,12 +26,12 @@ class Differ {
     /**
      * Calculate diff (edit script) from $old to $new.
      *
-     * @param T[] $old Original array
-     * @param T[] $new New array
+     * @param array $old Original array
+     * @param array $new New array
      *
      * @return DiffElem[] Diff (edit script)
      */
-    public function diff(array $old, array $new): array {
+    public function diff(array $old, array $new) {
         list($trace, $x, $y) = $this->calculateTrace($old, $new);
         return $this->extractDiff($trace, $x, $y, $old, $new);
     }
@@ -43,23 +42,18 @@ class Differ {
      * If a sequence of remove operations is followed by the same number of add operations, these
      * will be coalesced into replace operations.
      *
-     * @param T[] $old Original array
-     * @param T[] $new New array
+     * @param array $old Original array
+     * @param array $new New array
      *
      * @return DiffElem[] Diff (edit script), including replace operations
      */
-    public function diffWithReplacements(array $old, array $new): array {
+    public function diffWithReplacements(array $old, array $new) {
         return $this->coalesceReplacements($this->diff($old, $new));
     }
 
-    /**
-     * @param T[] $old
-     * @param T[] $new
-     * @return array{array<int, array<int, int>>, int, int}
-     */
-    private function calculateTrace(array $old, array $new): array {
-        $n = \count($old);
-        $m = \count($new);
+    private function calculateTrace(array $a, array $b) {
+        $n = \count($a);
+        $m = \count($b);
         $max = $n + $m;
         $v = [1 => 0];
         $trace = [];
@@ -73,7 +67,7 @@ class Differ {
                 }
 
                 $y = $x - $k;
-                while ($x < $n && $y < $m && ($this->isEqual)($old[$x], $new[$y])) {
+                while ($x < $n && $y < $m && ($this->isEqual)($a[$x], $b[$y])) {
                     $x++;
                     $y++;
                 }
@@ -87,15 +81,7 @@ class Differ {
         throw new \Exception('Should not happen');
     }
 
-    /**
-     * @param array<int, array<int, int>> $trace
-     * @param int $x
-     * @param int $y
-     * @param T[] $old
-     * @param T[] $new
-     * @return DiffElem[]
-     */
-    private function extractDiff(array $trace, int $x, int $y, array $old, array $new): array {
+    private function extractDiff(array $trace, int $x, int $y, array $a, array $b) {
         $result = [];
         for ($d = \count($trace) - 1; $d >= 0; $d--) {
             $v = $trace[$d];
@@ -111,7 +97,7 @@ class Differ {
             $prevY = $prevX - $prevK;
 
             while ($x > $prevX && $y > $prevY) {
-                $result[] = new DiffElem(DiffElem::TYPE_KEEP, $old[$x-1], $new[$y-1]);
+                $result[] = new DiffElem(DiffElem::TYPE_KEEP, $a[$x-1], $b[$y-1]);
                 $x--;
                 $y--;
             }
@@ -121,12 +107,12 @@ class Differ {
             }
 
             while ($x > $prevX) {
-                $result[] = new DiffElem(DiffElem::TYPE_REMOVE, $old[$x-1], null);
+                $result[] = new DiffElem(DiffElem::TYPE_REMOVE, $a[$x-1], null);
                 $x--;
             }
 
             while ($y > $prevY) {
-                $result[] = new DiffElem(DiffElem::TYPE_ADD, null, $new[$y-1]);
+                $result[] = new DiffElem(DiffElem::TYPE_ADD, null, $b[$y-1]);
                 $y--;
             }
         }
@@ -139,7 +125,7 @@ class Differ {
      * @param DiffElem[] $diff
      * @return DiffElem[]
      */
-    private function coalesceReplacements(array $diff): array {
+    private function coalesceReplacements(array $diff) {
         $newDiff = [];
         $c = \count($diff);
         for ($i = 0; $i < $c; $i++) {

lib/PhpParser/Internal/PrintableNewAnonClassNode.php

@@ -15,10 +15,13 @@ use PhpParser\Node\Expr;
  *
  * @internal
  */
-class PrintableNewAnonClassNode extends Expr {
+class PrintableNewAnonClassNode extends Expr
+{
     /** @var Node\AttributeGroup[] PHP attribute groups */
     public $attrGroups;
-    /** @var (Node\Arg|Node\VariadicPlaceholder)[] Arguments */
+    /** @var int Modifiers */
+    public $flags;
+    /** @var Node\Arg[] Arguments */
     public $args;
     /** @var null|Node\Name Name of extended class */
     public $extends;
@@ -27,42 +30,35 @@ class PrintableNewAnonClassNode extends Expr {
     /** @var Node\Stmt[] Statements */
     public $stmts;
 
-    /**
-     * @param Node\AttributeGroup[] $attrGroups PHP attribute groups
-     * @param (Node\Arg|Node\VariadicPlaceholder)[] $args Arguments
-     * @param Node\Name|null $extends Name of extended class
-     * @param Node\Name[] $implements Names of implemented interfaces
-     * @param Node\Stmt[] $stmts Statements
-     * @param array<string, mixed> $attributes Attributes
-     */
     public function __construct(
-        array $attrGroups, array $args, ?Node\Name $extends, array $implements,
+        array $attrGroups, int $flags, array $args, Node\Name $extends = null, array $implements,
         array $stmts, array $attributes
     ) {
         parent::__construct($attributes);
         $this->attrGroups = $attrGroups;
+        $this->flags = $flags;
         $this->args = $args;
         $this->extends = $extends;
         $this->implements = $implements;
         $this->stmts = $stmts;
     }
 
-    public static function fromNewNode(Expr\New_ $newNode): self {
+    public static function fromNewNode(Expr\New_ $newNode) {
         $class = $newNode->class;
         assert($class instanceof Node\Stmt\Class_);
         // We don't assert that $class->name is null here, to allow consumers to assign unique names
         // to anonymous classes for their own purposes. We simplify ignore the name here.
         return new self(
-            $class->attrGroups, $newNode->args, $class->extends, $class->implements,
+            $class->attrGroups, $class->flags, $newNode->args, $class->extends, $class->implements,
             $class->stmts, $newNode->getAttributes()
         );
     }
 
-    public function getType(): string {
+    public function getType() : string {
         return 'Expr_PrintableNewAnonClass';
     }
 
-    public function getSubNodeNames(): array {
-        return ['attrGroups', 'args', 'extends', 'implements', 'stmts'];
+    public function getSubNodeNames() : array {
+        return ['attrGroups', 'flags', 'args', 'extends', 'implements', 'stmts'];
     }
 }

lib/PhpParser/Internal/TokenPolyfill.php

@@ -1,272 +0,0 @@
-<?php declare(strict_types=1);
-
-namespace PhpParser\Internal;
-
-if (\PHP_VERSION_ID >= 80000) {
-    class TokenPolyfill extends \PhpToken {
-    }
-    return;
-}
-
-/**
- * This is a polyfill for the PhpToken class introduced in PHP 8.0. We do not actually polyfill
- * PhpToken, because composer might end up picking a different polyfill implementation, which does
- * not meet our requirements.
- *
- * @internal
- */
-class TokenPolyfill {
-    /** @var int The ID of the token. Either a T_* constant of a character code < 256. */
-    public $id;
-    /** @var string The textual content of the token. */
-    public $text;
-    /** @var int The 1-based starting line of the token (or -1 if unknown). */
-    public $line;
-    /** @var int The 0-based starting position of the token (or -1 if unknown). */
-    public $pos;
-
-    /** @var array<int, bool> Tokens ignored by the PHP parser. */
-    private const IGNORABLE_TOKENS = [
-        \T_WHITESPACE => true,
-        \T_COMMENT => true,
-        \T_DOC_COMMENT => true,
-        \T_OPEN_TAG => true,
-    ];
-
-    /** @var array<int, bool>|null Tokens that may be part of a T_NAME_* identifier. */
-    private static $identifierTokens;
-
-    /**
-     * Create a Token with the given ID and text, as well optional line and position information.
-     */
-    final public function __construct(int $id, string $text, int $line = -1, int $pos = -1) {
-        $this->id = $id;
-        $this->text = $text;
-        $this->line = $line;
-        $this->pos = $pos;
-    }
-
-    /**
-     * Get the name of the token. For single-char tokens this will be the token character.
-     * Otherwise it will be a T_* style name, or null if the token ID is unknown.
-     */
-    public function getTokenName(): ?string {
-        if ($this->id < 256) {
-            return \chr($this->id);
-        }
-
-        $name = token_name($this->id);
-        return $name === 'UNKNOWN' ? null : $name;
-    }
-
-    /**
-     * Check whether the token is of the given kind. The kind may be either an integer that matches
-     * the token ID, a string that matches the token text, or an array of integers/strings. In the
-     * latter case, the function returns true if any of the kinds in the array match.
-     *
-     * @param int|string|(int|string)[] $kind
-     */
-    public function is($kind): bool {
-        if (\is_int($kind)) {
-            return $this->id === $kind;
-        }
-        if (\is_string($kind)) {
-            return $this->text === $kind;
-        }
-        if (\is_array($kind)) {
-            foreach ($kind as $entry) {
-                if (\is_int($entry)) {
-                    if ($this->id === $entry) {
-                        return true;
-                    }
-                } elseif (\is_string($entry)) {
-                    if ($this->text === $entry) {
-                        return true;
-                    }
-                } else {
-                    throw new \TypeError(
-                        'Argument #1 ($kind) must only have elements of type string|int, ' .
-                        gettype($entry) . ' given');
-                }
-            }
-            return false;
-        }
-        throw new \TypeError(
-            'Argument #1 ($kind) must be of type string|int|array, ' .gettype($kind) . ' given');
-    }
-
-    /**
-     * Check whether this token would be ignored by the PHP parser. Returns true for T_WHITESPACE,
-     * T_COMMENT, T_DOC_COMMENT and T_OPEN_TAG, and false for everything else.
-     */
-    public function isIgnorable(): bool {
-        return isset(self::IGNORABLE_TOKENS[$this->id]);
-    }
-
-    /**
-     * Return the textual content of the token.
-     */
-    public function __toString(): string {
-        return $this->text;
-    }
-
-    /**
-     * Tokenize the given source code and return an array of tokens.
-     *
-     * This performs certain canonicalizations to match the PHP 8.0 token format:
-     *  * Bad characters are represented using T_BAD_CHARACTER rather than omitted.
-     *  * T_COMMENT does not include trailing newlines, instead the newline is part of a following
-     *    T_WHITESPACE token.
-     *  * Namespaced names are represented using T_NAME_* tokens.
-     *
-     * @return static[]
-     */
-    public static function tokenize(string $code, int $flags = 0): array {
-        self::init();
-
-        $tokens = [];
-        $line = 1;
-        $pos = 0;
-        $origTokens = \token_get_all($code, $flags);
-        if (\PHP_VERSION_ID < 70400) {
-            $origTokens = self::fixupBadCharacters($code, $origTokens);
-        }
-
-        $numTokens = \count($origTokens);
-        for ($i = 0; $i < $numTokens; $i++) {
-            $token = $origTokens[$i];
-            if (\is_string($token)) {
-                if (\strlen($token) === 2) {
-                    // b" and B" are tokenized as single-char tokens, even though they aren't.
-                    $tokens[] = new static(\ord('"'), $token, $line, $pos);
-                    $pos += 2;
-                } else {
-                    $tokens[] = new static(\ord($token), $token, $line, $pos);
-                    $pos++;
-                }
-            } else {
-                $id = $token[0];
-                $text = $token[1];
-
-                // Emulate PHP 8.0 comment format, which does not include trailing whitespace anymore.
-                if ($id === \T_COMMENT && \substr($text, 0, 2) !== '/*' &&
-                    \preg_match('/(\r\n|\n|\r)$/D', $text, $matches)
-                ) {
-                    $trailingNewline = $matches[0];
-                    $text = \substr($text, 0, -\strlen($trailingNewline));
-                    $tokens[] = new static($id, $text, $line, $pos);
-                    $pos += \strlen($text);
-
-                    if ($i + 1 < $numTokens && $origTokens[$i + 1][0] === \T_WHITESPACE) {
-                        // Move trailing newline into following T_WHITESPACE token, if it already exists.
-                        $origTokens[$i + 1][1] = $trailingNewline . $origTokens[$i + 1][1];
-                        $origTokens[$i + 1][2]--;
-                    } else {
-                        // Otherwise, we need to create a new T_WHITESPACE token.
-                        $tokens[] = new static(\T_WHITESPACE, $trailingNewline, $line, $pos);
-                        $line++;
-                        $pos += \strlen($trailingNewline);
-                    }
-                    continue;
-                }
-
-                // Emulate PHP 8.0 T_NAME_* tokens, by combining sequences of T_NS_SEPARATOR and
-                // T_STRING into a single token.
-                if (($id === \T_NS_SEPARATOR || isset(self::$identifierTokens[$id]))) {
-                    $newText = $text;
-                    $lastWasSeparator = $id === \T_NS_SEPARATOR;
-                    for ($j = $i + 1; $j < $numTokens; $j++) {
-                        if ($lastWasSeparator) {
-                            if (!isset(self::$identifierTokens[$origTokens[$j][0]])) {
-                                break;
-                            }
-                            $lastWasSeparator = false;
-                        } else {
-                            if ($origTokens[$j][0] !== \T_NS_SEPARATOR) {
-                                break;
-                            }
-                            $lastWasSeparator = true;
-                        }
-                        $newText .= $origTokens[$j][1];
-                    }
-                    if ($lastWasSeparator) {
-                        // Trailing separator is not part of the name.
-                        $j--;
-                        $newText = \substr($newText, 0, -1);
-                    }
-                    if ($j > $i + 1) {
-                        if ($id === \T_NS_SEPARATOR) {
-                            $id = \T_NAME_FULLY_QUALIFIED;
-                        } elseif ($id === \T_NAMESPACE) {
-                            $id = \T_NAME_RELATIVE;
-                        } else {
-                            $id = \T_NAME_QUALIFIED;
-                        }
-                        $tokens[] = new static($id, $newText, $line, $pos);
-                        $pos += \strlen($newText);
-                        $i = $j - 1;
-                        continue;
-                    }
-                }
-
-                $tokens[] = new static($id, $text, $line, $pos);
-                $line += \substr_count($text, "\n");
-                $pos += \strlen($text);
-            }
-        }
-        return $tokens;
-    }
-
-    /**
-     * Prior to PHP 7.4, token_get_all() simply dropped invalid characters from the token stream.
-     * Detect such cases and replace them with T_BAD_CHARACTER.
-     */
-    private static function fixupBadCharacters(string $code, array $origTokens): array {
-        $newTokens = [];
-        $pos = 0;
-        foreach ($origTokens as $token) {
-            $text = \is_string($token) ? $token : $token[1];
-            $len = \strlen($text);
-            if (substr($code, $pos, $len) !== $text) {
-                $nextPos = strpos($code, $text, $pos);
-                for ($i = $pos; $i < $nextPos; $i++) {
-                    // Don't bother including the line, we're not going to use it anyway.
-                    $newTokens[] = [\T_BAD_CHARACTER, $code[$i]];
-                }
-                $pos = $nextPos;
-            }
-            $pos += $len;
-            $newTokens[] = $token;
-        }
-
-        // Handle trailing invalid characters.
-        $codeLen = \strlen($code);
-        if ($pos !== $codeLen) {
-            for ($i = $pos; $i < $codeLen; $i++) {
-                $newTokens[] = [\T_BAD_CHARACTER, $code[$i]];
-            }
-        }
-        return $newTokens;
-    }
-
-    /** Initialize private static state needed by tokenize(). */
-    private static function init(): void {
-        if (isset(self::$identifierTokens)) {
-            return;
-        }
-
-        // Based on semi_reserved production.
-        self::$identifierTokens = \array_fill_keys([
-            \T_STRING,
-            \T_STATIC, \T_ABSTRACT, \T_FINAL, \T_PRIVATE, \T_PROTECTED, \T_PUBLIC, \T_READONLY,
-            \T_INCLUDE, \T_INCLUDE_ONCE, \T_EVAL, \T_REQUIRE, \T_REQUIRE_ONCE, \T_LOGICAL_OR, \T_LOGICAL_XOR, \T_LOGICAL_AND,
-            \T_INSTANCEOF, \T_NEW, \T_CLONE, \T_EXIT, \T_IF, \T_ELSEIF, \T_ELSE, \T_ENDIF, \T_ECHO, \T_DO, \T_WHILE,
-            \T_ENDWHILE, \T_FOR, \T_ENDFOR, \T_FOREACH, \T_ENDFOREACH, \T_DECLARE, \T_ENDDECLARE, \T_AS, \T_TRY, \T_CATCH,
-            \T_FINALLY, \T_THROW, \T_USE, \T_INSTEADOF, \T_GLOBAL, \T_VAR, \T_UNSET, \T_ISSET, \T_EMPTY, \T_CONTINUE, \T_GOTO,
-            \T_FUNCTION, \T_CONST, \T_RETURN, \T_PRINT, \T_YIELD, \T_LIST, \T_SWITCH, \T_ENDSWITCH, \T_CASE, \T_DEFAULT,
-            \T_BREAK, \T_ARRAY, \T_CALLABLE, \T_EXTENDS, \T_IMPLEMENTS, \T_NAMESPACE, \T_TRAIT, \T_INTERFACE, \T_CLASS,
-            \T_CLASS_C, \T_TRAIT_C, \T_FUNC_C, \T_METHOD_C, \T_LINE, \T_FILE, \T_DIR, \T_NS_C, \T_HALT_COMPILER, \T_FN,
-            \T_MATCH,
-        ], true);
-    }
-}

lib/PhpParser/Internal/TokenStream.php

@@ -2,15 +2,14 @@
 
 namespace PhpParser\Internal;
 
-use PhpParser\Token;
-
 /**
  * Provides operations on token streams, for use by pretty printer.
  *
  * @internal
  */
-class TokenStream {
-    /** @var Token[] Tokens (in PhpToken::tokenize() format) */
+class TokenStream
+{
+    /** @var array Tokens (in token_get_all format) */
     private $tokens;
     /** @var int[] Map from position to indentation */
     private $indentMap;
@@ -18,7 +17,7 @@ class TokenStream {
     /**
      * Create token stream instance.
      *
-     * @param Token[] $tokens Tokens in PhpToken::tokenize() format
+     * @param array $tokens Tokens in token_get_all() format
      */
     public function __construct(array $tokens) {
         $this->tokens = $tokens;
@@ -33,7 +32,7 @@ class TokenStream {
      *
      * @return bool
      */
-    public function haveParens(int $startPos, int $endPos): bool {
+    public function haveParens(int $startPos, int $endPos) : bool {
         return $this->haveTokenImmediatelyBefore($startPos, '(')
             && $this->haveTokenImmediatelyAfter($endPos, ')');
     }
@@ -46,7 +45,7 @@ class TokenStream {
      *
      * @return bool
      */
-    public function haveBraces(int $startPos, int $endPos): bool {
+    public function haveBraces(int $startPos, int $endPos) : bool {
         return ($this->haveTokenImmediatelyBefore($startPos, '{')
                 || $this->haveTokenImmediatelyBefore($startPos, T_CURLY_OPEN))
             && $this->haveTokenImmediatelyAfter($endPos, '}');
@@ -62,15 +61,16 @@ class TokenStream {
      *
      * @return bool Whether the expected token was found
      */
-    public function haveTokenImmediatelyBefore(int $pos, $expectedTokenType): bool {
+    public function haveTokenImmediatelyBefore(int $pos, $expectedTokenType) : bool {
         $tokens = $this->tokens;
         $pos--;
         for (; $pos >= 0; $pos--) {
-            $token = $tokens[$pos];
-            if ($token->is($expectedTokenType)) {
+            $tokenType = $tokens[$pos][0];
+            if ($tokenType === $expectedTokenType) {
                 return true;
             }
-            if (!$token->isIgnorable()) {
+            if ($tokenType !== \T_WHITESPACE
+                && $tokenType !== \T_COMMENT && $tokenType !== \T_DOC_COMMENT) {
                 break;
             }
         }
@@ -87,23 +87,23 @@ class TokenStream {
      *
      * @return bool Whether the expected token was found
      */
-    public function haveTokenImmediatelyAfter(int $pos, $expectedTokenType): bool {
+    public function haveTokenImmediatelyAfter(int $pos, $expectedTokenType) : bool {
         $tokens = $this->tokens;
         $pos++;
-        for ($c = \count($tokens); $pos < $c; $pos++) {
-            $token = $tokens[$pos];
-            if ($token->is($expectedTokenType)) {
+        for (; $pos < \count($tokens); $pos++) {
+            $tokenType = $tokens[$pos][0];
+            if ($tokenType === $expectedTokenType) {
                 return true;
             }
-            if (!$token->isIgnorable()) {
+            if ($tokenType !== \T_WHITESPACE
+                && $tokenType !== \T_COMMENT && $tokenType !== \T_DOC_COMMENT) {
                 break;
             }
         }
         return false;
     }
 
-    /** @param int|string|(int|string)[] $skipTokenType */
-    public function skipLeft(int $pos, $skipTokenType): int {
+    public function skipLeft(int $pos, $skipTokenType) {
         $tokens = $this->tokens;
 
         $pos = $this->skipLeftWhitespace($pos);
@@ -111,7 +111,7 @@ class TokenStream {
             return $pos;
         }
 
-        if (!$tokens[$pos]->is($skipTokenType)) {
+        if ($tokens[$pos][0] !== $skipTokenType) {
             // Shouldn't happen. The skip token MUST be there
             throw new \Exception('Encountered unexpected token');
         }
@@ -120,8 +120,7 @@ class TokenStream {
         return $this->skipLeftWhitespace($pos);
     }
 
-    /** @param int|string|(int|string)[] $skipTokenType */
-    public function skipRight(int $pos, $skipTokenType): int {
+    public function skipRight(int $pos, $skipTokenType) {
         $tokens = $this->tokens;
 
         $pos = $this->skipRightWhitespace($pos);
@@ -129,7 +128,7 @@ class TokenStream {
             return $pos;
         }
 
-        if (!$tokens[$pos]->is($skipTokenType)) {
+        if ($tokens[$pos][0] !== $skipTokenType) {
             // Shouldn't happen. The skip token MUST be there
             throw new \Exception('Encountered unexpected token');
         }
@@ -144,10 +143,11 @@ class TokenStream {
      * @param int $pos Token position
      * @return int Non-whitespace token position
      */
-    public function skipLeftWhitespace(int $pos): int {
+    public function skipLeftWhitespace(int $pos) {
         $tokens = $this->tokens;
         for (; $pos >= 0; $pos--) {
-            if (!$tokens[$pos]->isIgnorable()) {
+            $type = $tokens[$pos][0];
+            if ($type !== \T_WHITESPACE && $type !== \T_COMMENT && $type !== \T_DOC_COMMENT) {
                 break;
             }
         }
@@ -160,21 +160,22 @@ class TokenStream {
      * @param int $pos Token position
      * @return int Non-whitespace token position
      */
-    public function skipRightWhitespace(int $pos): int {
+    public function skipRightWhitespace(int $pos) {
         $tokens = $this->tokens;
         for ($count = \count($tokens); $pos < $count; $pos++) {
-            if (!$tokens[$pos]->isIgnorable()) {
+            $type = $tokens[$pos][0];
+            if ($type !== \T_WHITESPACE && $type !== \T_COMMENT && $type !== \T_DOC_COMMENT) {
                 break;
             }
         }
         return $pos;
     }
 
-    /** @param int|string|(int|string)[] $findTokenType */
-    public function findRight(int $pos, $findTokenType): int {
+    public function findRight(int $pos, $findTokenType) {
         $tokens = $this->tokens;
         for ($count = \count($tokens); $pos < $count; $pos++) {
-            if ($tokens[$pos]->is($findTokenType)) {
+            $type = $tokens[$pos][0];
+            if ($type === $findTokenType) {
                 return $pos;
             }
         }
@@ -189,17 +190,17 @@ class TokenStream {
      * @param int|string $tokenType Token type to look for
      * @return bool Whether the token occurs in the given range
      */
-    public function haveTokenInRange(int $startPos, int $endPos, $tokenType): bool {
+    public function haveTokenInRange(int $startPos, int $endPos, $tokenType) {
         $tokens = $this->tokens;
         for ($pos = $startPos; $pos < $endPos; $pos++) {
-            if ($tokens[$pos]->is($tokenType)) {
+            if ($tokens[$pos][0] === $tokenType) {
                 return true;
             }
         }
         return false;
     }
 
-    public function haveBracesInRange(int $startPos, int $endPos): bool {
+    public function haveBracesInRange(int $startPos, int $endPos) {
         return $this->haveTokenInRange($startPos, $endPos, '{')
             || $this->haveTokenInRange($startPos, $endPos, T_CURLY_OPEN)
             || $this->haveTokenInRange($startPos, $endPos, '}');
@@ -217,7 +218,7 @@ class TokenStream {
      *
      * @return int Indentation depth (in spaces)
      */
-    public function getIndentationBefore(int $pos): int {
+    public function getIndentationBefore(int $pos) : int {
         return $this->indentMap[$pos];
     }
 
@@ -230,24 +231,28 @@ class TokenStream {
      *
      * @return string Code corresponding to token range, adjusted for indentation
      */
-    public function getTokenCode(int $from, int $to, int $indent): string {
+    public function getTokenCode(int $from, int $to, int $indent) : string {
         $tokens = $this->tokens;
         $result = '';
         for ($pos = $from; $pos < $to; $pos++) {
             $token = $tokens[$pos];
-            $id = $token->id;
-            $text = $token->text;
-            if ($id === \T_CONSTANT_ENCAPSED_STRING || $id === \T_ENCAPSED_AND_WHITESPACE) {
-                $result .= $text;
-            } else {
-                // TODO Handle non-space indentation
-                if ($indent < 0) {
-                    $result .= str_replace("\n" . str_repeat(" ", -$indent), "\n", $text);
-                } elseif ($indent > 0) {
-                    $result .= str_replace("\n", "\n" . str_repeat(" ", $indent), $text);
+            if (\is_array($token)) {
+                $type = $token[0];
+                $content = $token[1];
+                if ($type === \T_CONSTANT_ENCAPSED_STRING || $type === \T_ENCAPSED_AND_WHITESPACE) {
+                    $result .= $content;
                 } else {
-                    $result .= $text;
+                    // TODO Handle non-space indentation
+                    if ($indent < 0) {
+                        $result .= str_replace("\n" . str_repeat(" ", -$indent), "\n", $content);
+                    } elseif ($indent > 0) {
+                        $result .= str_replace("\n", "\n" . str_repeat(" ", $indent), $content);
+                    } else {
+                        $result .= $content;
+                    }
                 }
+            } else {
+                $result .= $token;
             }
         }
         return $result;
@@ -258,14 +263,14 @@ class TokenStream {
      *
      * @return int[] Token position to indentation map
      */
-    private function calcIndentMap(): array {
+    private function calcIndentMap() {
         $indentMap = [];
         $indent = 0;
         foreach ($this->tokens as $token) {
             $indentMap[] = $indent;
 
-            if ($token->id === \T_WHITESPACE) {
-                $content = $token->text;
+            if ($token[0] === \T_WHITESPACE) {
+                $content = $token[1];
                 $newlinePos = \strrpos($content, "\n");
                 if (false !== $newlinePos) {
                     $indent = \strlen($content) - $newlinePos - 1;

lib/PhpParser/JsonDecoder.php

@@ -2,11 +2,11 @@
 
 namespace PhpParser;
 
-class JsonDecoder {
-    /** @var \ReflectionClass<Node>[] Node type to reflection class map */
+class JsonDecoder
+{
+    /** @var \ReflectionClass[] Node type to reflection class map */
     private $reflectionClassCache;
 
-    /** @return mixed */
     public function decode(string $json) {
         $value = json_decode($json, true);
         if (json_last_error()) {
@@ -16,10 +16,6 @@ class JsonDecoder {
         return $this->decodeRecursive($value);
     }
 
-    /**
-     * @param mixed $value
-     * @return mixed
-     */
     private function decodeRecursive($value) {
         if (\is_array($value)) {
             if (isset($value['nodeType'])) {
@@ -33,7 +29,7 @@ class JsonDecoder {
         return $value;
     }
 
-    private function decodeArray(array $array): array {
+    private function decodeArray(array $array) : array {
         $decodedArray = [];
         foreach ($array as $key => $value) {
             $decodedArray[$key] = $this->decodeRecursive($value);
@@ -41,13 +37,14 @@ class JsonDecoder {
         return $decodedArray;
     }
 
-    private function decodeNode(array $value): Node {
+    private function decodeNode(array $value) : Node {
         $nodeType = $value['nodeType'];
         if (!\is_string($nodeType)) {
             throw new \RuntimeException('Node type must be a string');
         }
 
         $reflectionClass = $this->reflectionClassFromNodeType($nodeType);
+        /** @var Node $node */
         $node = $reflectionClass->newInstanceWithoutConstructor();
 
         if (isset($value['attributes'])) {
@@ -69,7 +66,7 @@ class JsonDecoder {
         return $node;
     }
 
-    private function decodeComment(array $value): Comment {
+    private function decodeComment(array $value) : Comment {
         $className = $value['nodeType'] === 'Comment' ? Comment::class : Comment\Doc::class;
         if (!isset($value['text'])) {
             throw new \RuntimeException('Comment must have text');
@@ -82,8 +79,7 @@ class JsonDecoder {
         );
     }
 
-    /** @return \ReflectionClass<Node> */
-    private function reflectionClassFromNodeType(string $nodeType): \ReflectionClass {
+    private function reflectionClassFromNodeType(string $nodeType) : \ReflectionClass {
         if (!isset($this->reflectionClassCache[$nodeType])) {
             $className = $this->classNameFromNodeType($nodeType);
             $this->reflectionClassCache[$nodeType] = new \ReflectionClass($className);
@@ -91,8 +87,7 @@ class JsonDecoder {
         return $this->reflectionClassCache[$nodeType];
     }
 
-    /** @return class-string<Node> */
-    private function classNameFromNodeType(string $nodeType): string {
+    private function classNameFromNodeType(string $nodeType) : string {
         $className = 'PhpParser\\Node\\' . strtr($nodeType, '_', '\\');
         if (class_exists($className)) {
             return $className;

lib/PhpParser/Lexer.php

@@ -2,45 +2,44 @@
 
 namespace PhpParser;
 
-require __DIR__ . '/compatibility_tokens.php';
+use PhpParser\Parser\Tokens;
 
-class Lexer {
-    /** @var string Code being tokenized */
+class Lexer
+{
     protected $code;
-    /** @var list<Token> List of tokens */
     protected $tokens;
-    /** @var int Current position in the token array */
     protected $pos;
-    /** @var bool Whether the preceding closing PHP tag has a trailing newline */
+    protected $line;
+    protected $filePos;
     protected $prevCloseTagHasNewline;
-    /** @var array<int, int> Map of tokens that should be dropped (like T_WHITESPACE) */
-    protected $dropTokens;
 
-    /** @var bool Whether to use the startLine attribute */
+    protected $tokenMap;
+    protected $dropTokens;
+    protected $identifierTokens;
+
     private $attributeStartLineUsed;
-    /** @var bool Whether to use the endLine attribute */
     private $attributeEndLineUsed;
-    /** @var bool Whether to use the startTokenPos attribute */
     private $attributeStartTokenPosUsed;
-    /** @var bool Whether to use the endTokenPos attribute */
     private $attributeEndTokenPosUsed;
-    /** @var bool Whether to use the startFilePos attribute */
     private $attributeStartFilePosUsed;
-    /** @var bool Whether to use the endFilePos attribute */
     private $attributeEndFilePosUsed;
-    /** @var bool Whether to use the comments attribute */
     private $attributeCommentsUsed;
 
     /**
      * Creates a Lexer.
      *
-     * @param array{usedAttributes?: string[]} $options Options array. Currently only the
-     *        'usedAttributes' option is supported, which is an array of attributes to add to the
-     *        AST nodes. Possible attributes are: 'comments', 'startLine', 'endLine', 'startTokenPos',
-     *        'endTokenPos', 'startFilePos', 'endFilePos'. The option defaults to the first three.
-     *        For more info see getNextToken() docs.
+     * @param array $options Options array. Currently only the 'usedAttributes' option is supported,
+     *                       which is an array of attributes to add to the AST nodes. Possible
+     *                       attributes are: 'comments', 'startLine', 'endLine', 'startTokenPos',
+     *                       'endTokenPos', 'startFilePos', 'endFilePos'. The option defaults to the
+     *                       first three. For more info see getNextToken() docs.
      */
     public function __construct(array $options = []) {
+        // Create Map from internal tokens to PhpParser tokens.
+        $this->defineCompatibilityTokens();
+        $this->tokenMap = $this->createTokenMap();
+        $this->identifierTokens = $this->createIdentifierTokenMap();
+
         // map of tokens to drop while lexing (the map is only used for isset lookup,
         // that's why the value is simply set to 1; the value is never actually used.)
         $this->dropTokens = array_fill_keys(
@@ -70,13 +69,15 @@ class Lexer {
      * @param ErrorHandler|null $errorHandler Error handler to use for lexing errors. Defaults to
      *                                        ErrorHandler\Throwing
      */
-    public function startLexing(string $code, ?ErrorHandler $errorHandler = null): void {
+    public function startLexing(string $code, ErrorHandler $errorHandler = null) {
         if (null === $errorHandler) {
             $errorHandler = new ErrorHandler\Throwing();
         }
 
         $this->code = $code; // keep the code around for __halt_compiler() handling
-        $this->pos = -1;
+        $this->pos  = -1;
+        $this->line =  1;
+        $this->filePos = 0;
 
         // If inline HTML occurs without preceding code, treat it as if it had a leading newline.
         // This ensures proper composability, because having a newline is the "safe" assumption.
@@ -84,7 +85,7 @@ class Lexer {
 
         $scream = ini_set('xdebug.scream', '0');
 
-        $this->tokens = @Token::tokenize($code);
+        $this->tokens = @token_get_all($code);
         $this->postprocessTokens($errorHandler);
 
         if (false !== $scream) {
@@ -92,77 +93,192 @@ class Lexer {
         }
     }
 
-    private function handleInvalidCharacter(Token $token, ErrorHandler $errorHandler): void {
-        $chr = $token->text;
-        if ($chr === "\0") {
-            // PHP cuts error message after null byte, so need special case
-            $errorMsg = 'Unexpected null byte';
-        } else {
-            $errorMsg = sprintf(
-                'Unexpected character "%s" (ASCII %d)', $chr, ord($chr)
-            );
-        }
-
-        $errorHandler->handleError(new Error($errorMsg, [
-            'startLine' => $token->line,
-            'endLine' => $token->line,
-            'startFilePos' => $token->pos,
-            'endFilePos' => $token->pos,
-        ]));
-    }
-
-    private function isUnterminatedComment(Token $token): bool {
-        return $token->is([\T_COMMENT, \T_DOC_COMMENT])
-            && substr($token->text, 0, 2) === '/*'
-            && substr($token->text, -2) !== '*/';
-    }
-
-    protected function postprocessTokens(ErrorHandler $errorHandler): void {
-        // This function reports errors (bad characters and unterminated comments) in the token
-        // array, and performs certain canonicalizations:
-        //  * Use PHP 8.1 T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG and
-        //    T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG tokens used to disambiguate intersection types.
-        //  * Add a sentinel token with ID 0.
-
-        $numTokens = \count($this->tokens);
-        if ($numTokens === 0) {
-            // Empty input edge case: Just add the sentinel token.
-            $this->tokens[] = new Token(0, "\0", 1, 0);
-            return;
-        }
-
-        for ($i = 0; $i < $numTokens; $i++) {
-            $token = $this->tokens[$i];
-            if ($token->id === \T_BAD_CHARACTER) {
-                $this->handleInvalidCharacter($token, $errorHandler);
+    private function handleInvalidCharacterRange($start, $end, $line, ErrorHandler $errorHandler) {
+        $tokens = [];
+        for ($i = $start; $i < $end; $i++) {
+            $chr = $this->code[$i];
+            if ($chr === "\0") {
+                // PHP cuts error message after null byte, so need special case
+                $errorMsg = 'Unexpected null byte';
+            } else {
+                $errorMsg = sprintf(
+                    'Unexpected character "%s" (ASCII %d)', $chr, ord($chr)
+                );
             }
 
-            if ($token->id === \ord('&')) {
+            $tokens[] = [\T_BAD_CHARACTER, $chr, $line];
+            $errorHandler->handleError(new Error($errorMsg, [
+                'startLine' => $line,
+                'endLine' => $line,
+                'startFilePos' => $i,
+                'endFilePos' => $i,
+            ]));
+        }
+        return $tokens;
+    }
+
+    /**
+     * Check whether comment token is unterminated.
+     *
+     * @return bool
+     */
+    private function isUnterminatedComment($token) : bool {
+        return ($token[0] === \T_COMMENT || $token[0] === \T_DOC_COMMENT)
+            && substr($token[1], 0, 2) === '/*'
+            && substr($token[1], -2) !== '*/';
+    }
+
+    protected function postprocessTokens(ErrorHandler $errorHandler) {
+        // PHP's error handling for token_get_all() is rather bad, so if we want detailed
+        // error information we need to compute it ourselves. Invalid character errors are
+        // detected by finding "gaps" in the token array. Unterminated comments are detected
+        // by checking if a trailing comment has a "*/" at the end.
+        //
+        // Additionally, we perform a number of canonicalizations here:
+        //  * Use the PHP 8.0 comment format, which does not include trailing whitespace anymore.
+        //  * Use PHP 8.0 T_NAME_* tokens.
+        //  * Use PHP 8.1 T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG and
+        //    T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG tokens used to disambiguate intersection types.
+
+        $filePos = 0;
+        $line = 1;
+        $numTokens = \count($this->tokens);
+        for ($i = 0; $i < $numTokens; $i++) {
+            $token = $this->tokens[$i];
+
+            // Since PHP 7.4 invalid characters are represented by a T_BAD_CHARACTER token.
+            // In this case we only need to emit an error.
+            if ($token[0] === \T_BAD_CHARACTER) {
+                $this->handleInvalidCharacterRange($filePos, $filePos + 1, $line, $errorHandler);
+            }
+
+            if ($token[0] === \T_COMMENT && substr($token[1], 0, 2) !== '/*'
+                    && preg_match('/(\r\n|\n|\r)$/D', $token[1], $matches)) {
+                $trailingNewline = $matches[0];
+                $token[1] = substr($token[1], 0, -strlen($trailingNewline));
+                $this->tokens[$i] = $token;
+                if (isset($this->tokens[$i + 1]) && $this->tokens[$i + 1][0] === \T_WHITESPACE) {
+                    // Move trailing newline into following T_WHITESPACE token, if it already exists.
+                    $this->tokens[$i + 1][1] = $trailingNewline . $this->tokens[$i + 1][1];
+                    $this->tokens[$i + 1][2]--;
+                } else {
+                    // Otherwise, we need to create a new T_WHITESPACE token.
+                    array_splice($this->tokens, $i + 1, 0, [
+                        [\T_WHITESPACE, $trailingNewline, $line],
+                    ]);
+                    $numTokens++;
+                }
+            }
+
+            // Emulate PHP 8 T_NAME_* tokens, by combining sequences of T_NS_SEPARATOR and T_STRING
+            // into a single token.
+            if (\is_array($token)
+                    && ($token[0] === \T_NS_SEPARATOR || isset($this->identifierTokens[$token[0]]))) {
+                $lastWasSeparator = $token[0] === \T_NS_SEPARATOR;
+                $text = $token[1];
+                for ($j = $i + 1; isset($this->tokens[$j]); $j++) {
+                    if ($lastWasSeparator) {
+                        if (!isset($this->identifierTokens[$this->tokens[$j][0]])) {
+                            break;
+                        }
+                        $lastWasSeparator = false;
+                    } else {
+                        if ($this->tokens[$j][0] !== \T_NS_SEPARATOR) {
+                            break;
+                        }
+                        $lastWasSeparator = true;
+                    }
+                    $text .= $this->tokens[$j][1];
+                }
+                if ($lastWasSeparator) {
+                    // Trailing separator is not part of the name.
+                    $j--;
+                    $text = substr($text, 0, -1);
+                }
+                if ($j > $i + 1) {
+                    if ($token[0] === \T_NS_SEPARATOR) {
+                        $type = \T_NAME_FULLY_QUALIFIED;
+                    } else if ($token[0] === \T_NAMESPACE) {
+                        $type = \T_NAME_RELATIVE;
+                    } else {
+                        $type = \T_NAME_QUALIFIED;
+                    }
+                    $token = [$type, $text, $line];
+                    array_splice($this->tokens, $i, $j - $i, [$token]);
+                    $numTokens -= $j - $i - 1;
+                }
+            }
+
+            if ($token === '&') {
                 $next = $i + 1;
-                while (isset($this->tokens[$next]) && $this->tokens[$next]->id === \T_WHITESPACE) {
+                while (isset($this->tokens[$next]) && $this->tokens[$next][0] === \T_WHITESPACE) {
                     $next++;
                 }
                 $followedByVarOrVarArg = isset($this->tokens[$next]) &&
-                    $this->tokens[$next]->is([\T_VARIABLE, \T_ELLIPSIS]);
-                $token->id = $followedByVarOrVarArg
-                    ? \T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG
-                    : \T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG;
+                    ($this->tokens[$next][0] === \T_VARIABLE || $this->tokens[$next][0] === \T_ELLIPSIS);
+                $this->tokens[$i] = $token = [
+                    $followedByVarOrVarArg
+                        ? \T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG
+                        : \T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG,
+                    '&',
+                    $line,
+                ];
+            }
+
+            $tokenValue = \is_string($token) ? $token : $token[1];
+            $tokenLen = \strlen($tokenValue);
+
+            if (substr($this->code, $filePos, $tokenLen) !== $tokenValue) {
+                // Something is missing, must be an invalid character
+                $nextFilePos = strpos($this->code, $tokenValue, $filePos);
+                $badCharTokens = $this->handleInvalidCharacterRange(
+                    $filePos, $nextFilePos, $line, $errorHandler);
+                $filePos = (int) $nextFilePos;
+
+                array_splice($this->tokens, $i, 0, $badCharTokens);
+                $numTokens += \count($badCharTokens);
+                $i += \count($badCharTokens);
+            }
+
+            $filePos += $tokenLen;
+            $line += substr_count($tokenValue, "\n");
+        }
+
+        if ($filePos !== \strlen($this->code)) {
+            if (substr($this->code, $filePos, 2) === '/*') {
+                // Unlike PHP, HHVM will drop unterminated comments entirely
+                $comment = substr($this->code, $filePos);
+                $errorHandler->handleError(new Error('Unterminated comment', [
+                    'startLine' => $line,
+                    'endLine' => $line + substr_count($comment, "\n"),
+                    'startFilePos' => $filePos,
+                    'endFilePos' => $filePos + \strlen($comment),
+                ]));
+
+                // Emulate the PHP behavior
+                $isDocComment = isset($comment[3]) && $comment[3] === '*';
+                $this->tokens[] = [$isDocComment ? \T_DOC_COMMENT : \T_COMMENT, $comment, $line];
+            } else {
+                // Invalid characters at the end of the input
+                $badCharTokens = $this->handleInvalidCharacterRange(
+                    $filePos, \strlen($this->code), $line, $errorHandler);
+                $this->tokens = array_merge($this->tokens, $badCharTokens);
+            }
+            return;
+        }
+
+        if (count($this->tokens) > 0) {
+            // Check for unterminated comment
+            $lastToken = $this->tokens[count($this->tokens) - 1];
+            if ($this->isUnterminatedComment($lastToken)) {
+                $errorHandler->handleError(new Error('Unterminated comment', [
+                    'startLine' => $line - substr_count($lastToken[1], "\n"),
+                    'endLine' => $line,
+                    'startFilePos' => $filePos - \strlen($lastToken[1]),
+                    'endFilePos' => $filePos,
+                ]));
             }
         }
-
-        // Check for unterminated comment
-        $lastToken = $this->tokens[$numTokens - 1];
-        if ($this->isUnterminatedComment($lastToken)) {
-            $errorHandler->handleError(new Error('Unterminated comment', [
-                'startLine' => $lastToken->line,
-                'endLine' => $lastToken->getEndLine(),
-                'startFilePos' => $lastToken->pos,
-                'endFilePos' => $lastToken->getEndPos(),
-            ]));
-        }
-
-        // Add sentinel token.
-        $this->tokens[] = new Token(0, "\0", $lastToken->getEndLine(), $lastToken->getEndPos());
     }
 
     /**
@@ -187,76 +303,98 @@ class Lexer {
      *
      * @return int Token id
      */
-    public function getNextToken(&$value = null, &$startAttributes = null, &$endAttributes = null): int {
+    public function getNextToken(&$value = null, &$startAttributes = null, &$endAttributes = null) : int {
         $startAttributes = [];
         $endAttributes   = [];
 
         while (1) {
-            $token = $this->tokens[++$this->pos];
+            if (isset($this->tokens[++$this->pos])) {
+                $token = $this->tokens[$this->pos];
+            } else {
+                // EOF token with ID 0
+                $token = "\0";
+            }
 
             if ($this->attributeStartLineUsed) {
-                $startAttributes['startLine'] = $token->line;
+                $startAttributes['startLine'] = $this->line;
             }
             if ($this->attributeStartTokenPosUsed) {
                 $startAttributes['startTokenPos'] = $this->pos;
             }
             if ($this->attributeStartFilePosUsed) {
-                $startAttributes['startFilePos'] = $token->pos;
+                $startAttributes['startFilePos'] = $this->filePos;
             }
 
-            $id = $token->id;
-            if (isset($this->dropTokens[$id])) {
-                if (\T_COMMENT === $id || \T_DOC_COMMENT === $id) {
+            if (\is_string($token)) {
+                $value = $token;
+                if (isset($token[1])) {
+                    // bug in token_get_all
+                    $this->filePos += 2;
+                    $id = ord('"');
+                } else {
+                    $this->filePos += 1;
+                    $id = ord($token);
+                }
+            } elseif (!isset($this->dropTokens[$token[0]])) {
+                $value = $token[1];
+                $id = $this->tokenMap[$token[0]];
+                if (\T_CLOSE_TAG === $token[0]) {
+                    $this->prevCloseTagHasNewline = false !== strpos($token[1], "\n")
+                        || false !== strpos($token[1], "\r");
+                } elseif (\T_INLINE_HTML === $token[0]) {
+                    $startAttributes['hasLeadingNewline'] = $this->prevCloseTagHasNewline;
+                }
+
+                $this->line += substr_count($value, "\n");
+                $this->filePos += \strlen($value);
+            } else {
+                $origLine = $this->line;
+                $origFilePos = $this->filePos;
+                $this->line += substr_count($token[1], "\n");
+                $this->filePos += \strlen($token[1]);
+
+                if (\T_COMMENT === $token[0] || \T_DOC_COMMENT === $token[0]) {
                     if ($this->attributeCommentsUsed) {
-                        $comment = \T_DOC_COMMENT === $id
-                            ? new Comment\Doc($token->text, $token->line, $token->pos, $this->pos,
-                                $token->getEndLine(), $token->getEndPos() - 1, $this->pos)
-                            : new Comment($token->text, $token->line, $token->pos, $this->pos,
-                                $token->getEndLine(), $token->getEndPos() - 1, $this->pos);
+                        $comment = \T_DOC_COMMENT === $token[0]
+                            ? new Comment\Doc($token[1],
+                                $origLine, $origFilePos, $this->pos,
+                                $this->line, $this->filePos - 1, $this->pos)
+                            : new Comment($token[1],
+                                $origLine, $origFilePos, $this->pos,
+                                $this->line, $this->filePos - 1, $this->pos);
                         $startAttributes['comments'][] = $comment;
                     }
                 }
                 continue;
             }
 
-            $value = $token->text;
-            if (\T_CLOSE_TAG === $token->id) {
-                $this->prevCloseTagHasNewline = false !== strpos($value, "\n")
-                    || false !== strpos($value, "\r");
-            } elseif (\T_INLINE_HTML === $token->id) {
-                $startAttributes['hasLeadingNewline'] = $this->prevCloseTagHasNewline;
-            }
-
-            // Fetch the end line/pos from the next token (if available) instead of recomputing it.
-            $nextToken = $this->tokens[$this->pos + 1] ?? null;
             if ($this->attributeEndLineUsed) {
-                $endAttributes['endLine'] = $nextToken ? $nextToken->line : $token->getEndLine();
+                $endAttributes['endLine'] = $this->line;
             }
             if ($this->attributeEndTokenPosUsed) {
                 $endAttributes['endTokenPos'] = $this->pos;
             }
             if ($this->attributeEndFilePosUsed) {
-                $endAttributes['endFilePos'] = ($nextToken ? $nextToken->pos : $token->getEndPos()) - 1;
+                $endAttributes['endFilePos'] = $this->filePos - 1;
             }
 
             return $id;
         }
+
+        throw new \RuntimeException('Reached end of lexer loop');
     }
 
     /**
      * Returns the token array for current code.
      *
-     * The token array is in the same format as provided by the PhpToken::tokenize() method in
-     * PHP 8.0. The tokens are instances of PhpParser\Token, to abstract over a polyfill
-     * implementation in earlier PHP version.
+     * The token array is in the same format as provided by the
+     * token_get_all() function and does not discard tokens (i.e.
+     * whitespace and comments are included). The token position
+     * attributes are against this token array.
      *
-     * The token array is terminated by a sentinel token with token ID 0.
-     * The token array does not discard any tokens (i.e. whitespace and comments are included).
-     * The token position attributes are against this token array.
-     *
-     * @return Token[] Array of tokens
+     * @return array Array of tokens in token_get_all() format
      */
-    public function getTokens(): array {
+    public function getTokens() : array {
         return $this->tokens;
     }
 
@@ -265,12 +403,158 @@ class Lexer {
      *
      * @return string Remaining text
      */
-    public function handleHaltCompiler(): string {
-        // Prevent the lexer from returning any further tokens.
-        $nextToken = $this->tokens[$this->pos + 1];
-        $this->pos = \count($this->tokens) - 2;
+    public function handleHaltCompiler() : string {
+        // text after T_HALT_COMPILER, still including ();
+        $textAfter = substr($this->code, $this->filePos);
 
-        // Return text after __halt_compiler.
-        return $nextToken->id === \T_INLINE_HTML ? $nextToken->text : '';
+        // ensure that it is followed by ();
+        // this simplifies the situation, by not allowing any comments
+        // in between of the tokens.
+        if (!preg_match('~^\s*\(\s*\)\s*(?:;|\?>\r?\n?)~', $textAfter, $matches)) {
+            throw new Error('__HALT_COMPILER must be followed by "();"');
+        }
+
+        // prevent the lexer from returning any further tokens
+        $this->pos = count($this->tokens);
+
+        // return with (); removed
+        return substr($textAfter, strlen($matches[0]));
+    }
+
+    private function defineCompatibilityTokens() {
+        static $compatTokensDefined = false;
+        if ($compatTokensDefined) {
+            return;
+        }
+
+        $compatTokens = [
+            // PHP 7.4
+            'T_BAD_CHARACTER',
+            'T_FN',
+            'T_COALESCE_EQUAL',
+            // PHP 8.0
+            'T_NAME_QUALIFIED',
+            'T_NAME_FULLY_QUALIFIED',
+            'T_NAME_RELATIVE',
+            'T_MATCH',
+            'T_NULLSAFE_OBJECT_OPERATOR',
+            'T_ATTRIBUTE',
+            // PHP 8.1
+            'T_ENUM',
+            'T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG',
+            'T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG',
+            'T_READONLY',
+        ];
+
+        // PHP-Parser might be used together with another library that also emulates some or all
+        // of these tokens. Perform a sanity-check that all already defined tokens have been
+        // assigned a unique ID.
+        $usedTokenIds = [];
+        foreach ($compatTokens as $token) {
+            if (\defined($token)) {
+                $tokenId = \constant($token);
+                $clashingToken = $usedTokenIds[$tokenId] ?? null;
+                if ($clashingToken !== null) {
+                    throw new \Error(sprintf(
+                        'Token %s has same ID as token %s, ' .
+                        'you may be using a library with broken token emulation',
+                        $token, $clashingToken
+                    ));
+                }
+                $usedTokenIds[$tokenId] = $token;
+            }
+        }
+
+        // Now define any tokens that have not yet been emulated. Try to assign IDs from -1
+        // downwards, but skip any IDs that may already be in use.
+        $newTokenId = -1;
+        foreach ($compatTokens as $token) {
+            if (!\defined($token)) {
+                while (isset($usedTokenIds[$newTokenId])) {
+                    $newTokenId--;
+                }
+                \define($token, $newTokenId);
+                $newTokenId--;
+            }
+        }
+
+        $compatTokensDefined = true;
+    }
+
+    /**
+     * Creates the token map.
+     *
+     * The token map maps the PHP internal token identifiers
+     * to the identifiers used by the Parser. Additionally it
+     * maps T_OPEN_TAG_WITH_ECHO to T_ECHO and T_CLOSE_TAG to ';'.
+     *
+     * @return array The token map
+     */
+    protected function createTokenMap() : array {
+        $tokenMap = [];
+
+        // 256 is the minimum possible token number, as everything below
+        // it is an ASCII value
+        for ($i = 256; $i < 1000; ++$i) {
+            if (\T_DOUBLE_COLON === $i) {
+                // T_DOUBLE_COLON is equivalent to T_PAAMAYIM_NEKUDOTAYIM
+                $tokenMap[$i] = Tokens::T_PAAMAYIM_NEKUDOTAYIM;
+            } elseif(\T_OPEN_TAG_WITH_ECHO === $i) {
+                // T_OPEN_TAG_WITH_ECHO with dropped T_OPEN_TAG results in T_ECHO
+                $tokenMap[$i] = Tokens::T_ECHO;
+            } elseif(\T_CLOSE_TAG === $i) {
+                // T_CLOSE_TAG is equivalent to ';'
+                $tokenMap[$i] = ord(';');
+            } elseif ('UNKNOWN' !== $name = token_name($i)) {
+                if ('T_HASHBANG' === $name) {
+                    // HHVM uses a special token for #! hashbang lines
+                    $tokenMap[$i] = Tokens::T_INLINE_HTML;
+                } elseif (defined($name = Tokens::class . '::' . $name)) {
+                    // Other tokens can be mapped directly
+                    $tokenMap[$i] = constant($name);
+                }
+            }
+        }
+
+        // HHVM uses a special token for numbers that overflow to double
+        if (defined('T_ONUMBER')) {
+            $tokenMap[\T_ONUMBER] = Tokens::T_DNUMBER;
+        }
+        // HHVM also has a separate token for the __COMPILER_HALT_OFFSET__ constant
+        if (defined('T_COMPILER_HALT_OFFSET')) {
+            $tokenMap[\T_COMPILER_HALT_OFFSET] = Tokens::T_STRING;
+        }
+
+        // Assign tokens for which we define compatibility constants, as token_name() does not know them.
+        $tokenMap[\T_FN] = Tokens::T_FN;
+        $tokenMap[\T_COALESCE_EQUAL] = Tokens::T_COALESCE_EQUAL;
+        $tokenMap[\T_NAME_QUALIFIED] = Tokens::T_NAME_QUALIFIED;
+        $tokenMap[\T_NAME_FULLY_QUALIFIED] = Tokens::T_NAME_FULLY_QUALIFIED;
+        $tokenMap[\T_NAME_RELATIVE] = Tokens::T_NAME_RELATIVE;
+        $tokenMap[\T_MATCH] = Tokens::T_MATCH;
+        $tokenMap[\T_NULLSAFE_OBJECT_OPERATOR] = Tokens::T_NULLSAFE_OBJECT_OPERATOR;
+        $tokenMap[\T_ATTRIBUTE] = Tokens::T_ATTRIBUTE;
+        $tokenMap[\T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG] = Tokens::T_AMPERSAND_NOT_FOLLOWED_BY_VAR_OR_VARARG;
+        $tokenMap[\T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG] = Tokens::T_AMPERSAND_FOLLOWED_BY_VAR_OR_VARARG;
+        $tokenMap[\T_ENUM] = Tokens::T_ENUM;
+        $tokenMap[\T_READONLY] = Tokens::T_READONLY;
+
+        return $tokenMap;
+    }
+
+    private function createIdentifierTokenMap(): array {
+        // Based on semi_reserved production.
+        return array_fill_keys([
+            \T_STRING,
+            \T_STATIC, \T_ABSTRACT, \T_FINAL, \T_PRIVATE, \T_PROTECTED, \T_PUBLIC, \T_READONLY,
+            \T_INCLUDE, \T_INCLUDE_ONCE, \T_EVAL, \T_REQUIRE, \T_REQUIRE_ONCE, \T_LOGICAL_OR, \T_LOGICAL_XOR, \T_LOGICAL_AND,
+            \T_INSTANCEOF, \T_NEW, \T_CLONE, \T_EXIT, \T_IF, \T_ELSEIF, \T_ELSE, \T_ENDIF, \T_ECHO, \T_DO, \T_WHILE,
+            \T_ENDWHILE, \T_FOR, \T_ENDFOR, \T_FOREACH, \T_ENDFOREACH, \T_DECLARE, \T_ENDDECLARE, \T_AS, \T_TRY, \T_CATCH,
+            \T_FINALLY, \T_THROW, \T_USE, \T_INSTEADOF, \T_GLOBAL, \T_VAR, \T_UNSET, \T_ISSET, \T_EMPTY, \T_CONTINUE, \T_GOTO,
+            \T_FUNCTION, \T_CONST, \T_RETURN, \T_PRINT, \T_YIELD, \T_LIST, \T_SWITCH, \T_ENDSWITCH, \T_CASE, \T_DEFAULT,
+            \T_BREAK, \T_ARRAY, \T_CALLABLE, \T_EXTENDS, \T_IMPLEMENTS, \T_NAMESPACE, \T_TRAIT, \T_INTERFACE, \T_CLASS,
+            \T_CLASS_C, \T_TRAIT_C, \T_FUNC_C, \T_METHOD_C, \T_LINE, \T_FILE, \T_DIR, \T_NS_C, \T_HALT_COMPILER, \T_FN,
+            \T_MATCH,
+        ], true);
     }
 }

lib/PhpParser/Lexer/Emulative.php

@@ -18,32 +18,32 @@ use PhpParser\Lexer\TokenEmulator\ReadonlyFunctionTokenEmulator;
 use PhpParser\Lexer\TokenEmulator\ReadonlyTokenEmulator;
 use PhpParser\Lexer\TokenEmulator\ReverseEmulator;
 use PhpParser\Lexer\TokenEmulator\TokenEmulator;
-use PhpParser\PhpVersion;
 
-class Emulative extends Lexer {
-    /** @var array{int, string, string}[] Patches used to reverse changes introduced in the code */
+class Emulative extends Lexer
+{
+    const PHP_7_3 = '7.3dev';
+    const PHP_7_4 = '7.4dev';
+    const PHP_8_0 = '8.0dev';
+    const PHP_8_1 = '8.1dev';
+    const PHP_8_2 = '8.2dev';
+
+    /** @var mixed[] Patches used to reverse changes introduced in the code */
     private $patches = [];
 
-    /** @var list<TokenEmulator> */
+    /** @var TokenEmulator[] */
     private $emulators = [];
 
-    /** @var PhpVersion */
+    /** @var string */
     private $targetPhpVersion;
-    /** @var PhpVersion */
-    private $hostPhpVersion;
 
     /**
-     * @param array{usedAttributes?: string[], phpVersion?: PhpVersion|string} $options Lexer options.
-     *        In addition to the usual options, accepts a 'phpVersion' (PhpVersion object or string)
-     *        that specifies the version to emulate. Defaults to newest supported.
+     * @param mixed[] $options Lexer options. In addition to the usual options,
+     *                         accepts a 'phpVersion' string that specifies the
+     *                         version to emulate. Defaults to newest supported.
      */
-    public function __construct(array $options = []) {
-        $version = $options['phpVersion'] ?? PhpVersion::getNewestSupported();
-        if (!$version instanceof PhpVersion) {
-            $version = PhpVersion::fromString($version);
-        }
-        $this->targetPhpVersion = $version;
-        $this->hostPhpVersion = PhpVersion::getHostVersion();
+    public function __construct(array $options = [])
+    {
+        $this->targetPhpVersion = $options['phpVersion'] ?? Emulative::PHP_8_2;
         unset($options['phpVersion']);
 
         parent::__construct($options);
@@ -68,14 +68,14 @@ class Emulative extends Lexer {
             $emulatorPhpVersion = $emulator->getPhpVersion();
             if ($this->isForwardEmulationNeeded($emulatorPhpVersion)) {
                 $this->emulators[] = $emulator;
-            } elseif ($this->isReverseEmulationNeeded($emulatorPhpVersion)) {
+            } else if ($this->isReverseEmulationNeeded($emulatorPhpVersion)) {
                 $this->emulators[] = new ReverseEmulator($emulator);
             }
         }
     }
 
-    public function startLexing(string $code, ?ErrorHandler $errorHandler = null): void {
-        $emulators = array_filter($this->emulators, function ($emulator) use ($code) {
+    public function startLexing(string $code, ErrorHandler $errorHandler = null) {
+        $emulators = array_filter($this->emulators, function($emulator) use($code) {
             return $emulator->isEmulationNeeded($code);
         });
 
@@ -108,41 +108,61 @@ class Emulative extends Lexer {
         }
     }
 
-    private function isForwardEmulationNeeded(PhpVersion $emulatorPhpVersion): bool {
-        return $this->hostPhpVersion->older($emulatorPhpVersion)
-            && $this->targetPhpVersion->newerOrEqual($emulatorPhpVersion);
+    private function isForwardEmulationNeeded(string $emulatorPhpVersion): bool {
+        return version_compare(\PHP_VERSION, $emulatorPhpVersion, '<')
+            && version_compare($this->targetPhpVersion, $emulatorPhpVersion, '>=');
     }
 
-    private function isReverseEmulationNeeded(PhpVersion $emulatorPhpVersion): bool {
-        return $this->hostPhpVersion->newerOrEqual($emulatorPhpVersion)
-            && $this->targetPhpVersion->older($emulatorPhpVersion);
+    private function isReverseEmulationNeeded(string $emulatorPhpVersion): bool {
+        return version_compare(\PHP_VERSION, $emulatorPhpVersion, '>=')
+            && version_compare($this->targetPhpVersion, $emulatorPhpVersion, '<');
     }
 
-    private function sortPatches(): void {
+    private function sortPatches()
+    {
         // Patches may be contributed by different emulators.
         // Make sure they are sorted by increasing patch position.
-        usort($this->patches, function ($p1, $p2) {
+        usort($this->patches, function($p1, $p2) {
             return $p1[0] <=> $p2[0];
         });
     }
 
-    private function fixupTokens(): void {
+    private function fixupTokens()
+    {
         if (\count($this->patches) === 0) {
             return;
         }
 
         // Load first patch
         $patchIdx = 0;
+
         list($patchPos, $patchType, $patchText) = $this->patches[$patchIdx];
 
         // We use a manual loop over the tokens, because we modify the array on the fly
-        $posDelta = 0;
+        $pos = 0;
         for ($i = 0, $c = \count($this->tokens); $i < $c; $i++) {
             $token = $this->tokens[$i];
-            $pos = $token->pos;
-            $token->pos += $posDelta;
-            $localPosDelta = 0;
-            $len = \strlen($token->text);
+            if (\is_string($token)) {
+                if ($patchPos === $pos) {
+                    // Only support replacement for string tokens.
+                    assert($patchType === 'replace');
+                    $this->tokens[$i] = $patchText;
+
+                    // Fetch the next patch
+                    $patchIdx++;
+                    if ($patchIdx >= \count($this->patches)) {
+                        // No more patches, we're done
+                        return;
+                    }
+                    list($patchPos, $patchType, $patchText) = $this->patches[$patchIdx];
+                }
+
+                $pos += \strlen($token);
+                continue;
+            }
+
+            $len = \strlen($token[1]);
+            $posDelta = 0;
             while ($patchPos >= $pos && $patchPos < $pos + $len) {
                 $patchTextLen = \strlen($patchText);
                 if ($patchType === 'remove') {
@@ -153,21 +173,21 @@ class Emulative extends Lexer {
                         $c--;
                     } else {
                         // Remove from token string
-                        $token->text = substr_replace(
-                            $token->text, '', $patchPos - $pos + $localPosDelta, $patchTextLen
+                        $this->tokens[$i][1] = substr_replace(
+                            $token[1], '', $patchPos - $pos + $posDelta, $patchTextLen
                         );
-                        $localPosDelta -= $patchTextLen;
+                        $posDelta -= $patchTextLen;
                     }
                 } elseif ($patchType === 'add') {
                     // Insert into the token string
-                    $token->text = substr_replace(
-                        $token->text, $patchText, $patchPos - $pos + $localPosDelta, 0
+                    $this->tokens[$i][1] = substr_replace(
+                        $token[1], $patchText, $patchPos - $pos + $posDelta, 0
                     );
-                    $localPosDelta += $patchTextLen;
-                } elseif ($patchType === 'replace') {
+                    $posDelta += $patchTextLen;
+                } else if ($patchType === 'replace') {
                     // Replace inside the token string
-                    $token->text = substr_replace(
-                        $token->text, $patchText, $patchPos - $pos + $localPosDelta, $patchTextLen
+                    $this->tokens[$i][1] = substr_replace(
+                        $token[1], $patchText, $patchPos - $pos + $posDelta, $patchTextLen
                     );
                 } else {
                     assert(false);
@@ -176,16 +196,22 @@ class Emulative extends Lexer {
                 // Fetch the next patch
                 $patchIdx++;
                 if ($patchIdx >= \count($this->patches)) {
-                    // No more patches. However, we still need to adjust position.
-                    $patchPos = \PHP_INT_MAX;
-                    break;
+                    // No more patches, we're done
+                    return;
                 }
 
                 list($patchPos, $patchType, $patchText) = $this->patches[$patchIdx];
+
+                // Multiple patches may apply to the same token. Reload the current one to check
+                // If the new patch applies
+                $token = $this->tokens[$i];
             }
 
-            $posDelta += $localPosDelta;
+            $pos += $len;
         }
+
+        // A patch did not apply
+        assert(false);
     }
 
     /**
@@ -193,7 +219,7 @@ class Emulative extends Lexer {
      *
      * @param Error[] $errors
      */
-    private function fixupErrors(array $errors): void {
+    private function fixupErrors(array $errors) {
         foreach ($errors as $error) {
             $attrs = $error->getAttributes();
 
@@ -209,7 +235,7 @@ class Emulative extends Lexer {
                 if ($patchType === 'add') {
                     $posDelta += strlen($patchText);
                     $lineDelta += substr_count($patchText, "\n");
-                } elseif ($patchType === 'remove') {
+                } else if ($patchType === 'remove') {
                     $posDelta -= strlen($patchText);
                     $lineDelta -= substr_count($patchText, "\n");
                 }

lib/PhpParser/Lexer/TokenEmulator/AttributeEmulator.php

@@ -2,36 +2,43 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-use PhpParser\Token;
+use PhpParser\Lexer\Emulative;
 
-final class AttributeEmulator extends TokenEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 0);
+final class AttributeEmulator extends TokenEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_8_0;
     }
 
-    public function isEmulationNeeded(string $code): bool {
+    public function isEmulationNeeded(string $code) : bool
+    {
         return strpos($code, '#[') !== false;
     }
 
-    public function emulate(string $code, array $tokens): array {
+    public function emulate(string $code, array $tokens): array
+    {
         // We need to manually iterate and manage a count because we'll change
         // the tokens array on the way.
+        $line = 1;
         for ($i = 0, $c = count($tokens); $i < $c; ++$i) {
-            $token = $tokens[$i];
-            if ($token->text === '#' && isset($tokens[$i + 1]) && $tokens[$i + 1]->text === '[') {
+            if ($tokens[$i] === '#' && isset($tokens[$i + 1]) && $tokens[$i + 1] === '[') {
                 array_splice($tokens, $i, 2, [
-                    new Token(\T_ATTRIBUTE, '#[', $token->line, $token->pos),
+                    [\T_ATTRIBUTE, '#[', $line]
                 ]);
                 $c--;
                 continue;
             }
+            if (\is_array($tokens[$i])) {
+                $line += substr_count($tokens[$i][1], "\n");
+            }
         }
 
         return $tokens;
     }
 
-    public function reverseEmulate(string $code, array $tokens): array {
+    public function reverseEmulate(string $code, array $tokens): array
+    {
         // TODO
         return $tokens;
     }

lib/PhpParser/Lexer/TokenEmulator/CoaleseEqualTokenEmulator.php

@@ -2,38 +2,45 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-use PhpParser\Token;
+use PhpParser\Lexer\Emulative;
 
-final class CoaleseEqualTokenEmulator extends TokenEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(7, 4);
+final class CoaleseEqualTokenEmulator extends TokenEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_7_4;
     }
 
-    public function isEmulationNeeded(string $code): bool {
+    public function isEmulationNeeded(string $code): bool
+    {
         return strpos($code, '??=') !== false;
     }
 
-    public function emulate(string $code, array $tokens): array {
+    public function emulate(string $code, array $tokens): array
+    {
         // We need to manually iterate and manage a count because we'll change
         // the tokens array on the way
+        $line = 1;
         for ($i = 0, $c = count($tokens); $i < $c; ++$i) {
-            $token = $tokens[$i];
             if (isset($tokens[$i + 1])) {
-                if ($token->id === T_COALESCE && $tokens[$i + 1]->text === '=') {
+                if ($tokens[$i][0] === T_COALESCE && $tokens[$i + 1] === '=') {
                     array_splice($tokens, $i, 2, [
-                        new Token(\T_COALESCE_EQUAL, '??=', $token->line, $token->pos),
+                        [\T_COALESCE_EQUAL, '??=', $line]
                     ]);
                     $c--;
                     continue;
                 }
             }
+            if (\is_array($tokens[$i])) {
+                $line += substr_count($tokens[$i][1], "\n");
+            }
         }
 
         return $tokens;
     }
 
-    public function reverseEmulate(string $code, array $tokens): array {
+    public function reverseEmulate(string $code, array $tokens): array
+    {
         // ??= was not valid code previously, don't bother.
         return $tokens;
     }

lib/PhpParser/Lexer/TokenEmulator/EnumTokenEmulator.php

@@ -2,25 +2,30 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
+use PhpParser\Lexer\Emulative;
 
-final class EnumTokenEmulator extends KeywordEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 1);
+final class EnumTokenEmulator extends KeywordEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_8_1;
     }
 
-    public function getKeywordString(): string {
+    public function getKeywordString(): string
+    {
         return 'enum';
     }
 
-    public function getKeywordToken(): int {
+    public function getKeywordToken(): int
+    {
         return \T_ENUM;
     }
 
-    protected function isKeywordContext(array $tokens, int $pos): bool {
+    protected function isKeywordContext(array $tokens, int $pos): bool
+    {
         return parent::isKeywordContext($tokens, $pos)
             && isset($tokens[$pos + 2])
-            && $tokens[$pos + 1]->id === \T_WHITESPACE
-            && $tokens[$pos + 2]->id === \T_STRING;
+            && $tokens[$pos + 1][0] === \T_WHITESPACE
+            && $tokens[$pos + 2][0] === \T_STRING;
     }
-}
+}

lib/PhpParser/Lexer/TokenEmulator/ExplicitOctalEmulator.php

@@ -2,12 +2,11 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-use PhpParser\Token;
+use PhpParser\Lexer\Emulative;
 
 class ExplicitOctalEmulator extends TokenEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 1);
+    public function getPhpVersion(): string {
+        return Emulative::PHP_8_1;
     }
 
     public function isEmulationNeeded(string $code): bool {
@@ -16,14 +15,13 @@ class ExplicitOctalEmulator extends TokenEmulator {
 
     public function emulate(string $code, array $tokens): array {
         for ($i = 0, $c = count($tokens); $i < $c; ++$i) {
-            $token = $tokens[$i];
-            if ($token->id == \T_LNUMBER && $token->text === '0' &&
-                isset($tokens[$i + 1]) && $tokens[$i + 1]->id == \T_STRING &&
-                preg_match('/[oO][0-7]+(?:_[0-7]+)*/', $tokens[$i + 1]->text)
+            if ($tokens[$i][0] == \T_LNUMBER && $tokens[$i][1] === '0' &&
+                isset($tokens[$i + 1]) && $tokens[$i + 1][0] == \T_STRING &&
+                preg_match('/[oO][0-7]+(?:_[0-7]+)*/', $tokens[$i + 1][1])
             ) {
-                $tokenKind = $this->resolveIntegerOrFloatToken($tokens[$i + 1]->text);
+                $tokenKind = $this->resolveIntegerOrFloatToken($tokens[$i + 1][1]);
                 array_splice($tokens, $i, 2, [
-                    new Token($tokenKind, '0' . $tokens[$i + 1]->text, $token->line, $token->pos),
+                    [$tokenKind, '0' . $tokens[$i + 1][1], $tokens[$i][2]],
                 ]);
                 $c--;
             }
@@ -31,7 +29,8 @@ class ExplicitOctalEmulator extends TokenEmulator {
         return $tokens;
     }
 
-    private function resolveIntegerOrFloatToken(string $str): int {
+    private function resolveIntegerOrFloatToken(string $str): int
+    {
         $str = substr($str, 1);
         $str = str_replace('_', '', $str);
         $num = octdec($str);
@@ -42,4 +41,4 @@ class ExplicitOctalEmulator extends TokenEmulator {
         // Explicit octals were not legal code previously, don't bother.
         return $tokens;
     }
-}
+}

lib/PhpParser/Lexer/TokenEmulator/FlexibleDocStringEmulator.php

@@ -2,29 +2,34 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
+use PhpParser\Lexer\Emulative;
 
-final class FlexibleDocStringEmulator extends TokenEmulator {
-    private const FLEXIBLE_DOC_STRING_REGEX = <<<'REGEX'
+final class FlexibleDocStringEmulator extends TokenEmulator
+{
+    const FLEXIBLE_DOC_STRING_REGEX = <<<'REGEX'
 /<<<[ \t]*(['"]?)([a-zA-Z_\x80-\xff][a-zA-Z0-9_\x80-\xff]*)\1\r?\n
 (?:.*\r?\n)*?
 (?<indentation>\h*)\2(?![a-zA-Z0-9_\x80-\xff])(?<separator>(?:;?[\r\n])?)/x
 REGEX;
 
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(7, 3);
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_7_3;
     }
 
-    public function isEmulationNeeded(string $code): bool {
+    public function isEmulationNeeded(string $code) : bool
+    {
         return strpos($code, '<<<') !== false;
     }
 
-    public function emulate(string $code, array $tokens): array {
+    public function emulate(string $code, array $tokens): array
+    {
         // Handled by preprocessing + fixup.
         return $tokens;
     }
 
-    public function reverseEmulate(string $code, array $tokens): array {
+    public function reverseEmulate(string $code, array $tokens): array
+    {
         // Not supported.
         return $tokens;
     }

lib/PhpParser/Lexer/TokenEmulator/FnTokenEmulator.php

@@ -2,18 +2,22 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
+use PhpParser\Lexer\Emulative;
 
-final class FnTokenEmulator extends KeywordEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(7, 4);
+final class FnTokenEmulator extends KeywordEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_7_4;
     }
 
-    public function getKeywordString(): string {
+    public function getKeywordString(): string
+    {
         return 'fn';
     }
 
-    public function getKeywordToken(): int {
+    public function getKeywordToken(): int
+    {
         return \T_FN;
     }
-}
+}

lib/PhpParser/Lexer/TokenEmulator/KeywordEmulator.php

@@ -2,38 +2,43 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\Token;
+abstract class KeywordEmulator extends TokenEmulator
+{
+    abstract function getKeywordString(): string;
+    abstract function getKeywordToken(): int;
 
-abstract class KeywordEmulator extends TokenEmulator {
-    abstract public function getKeywordString(): string;
-    abstract public function getKeywordToken(): int;
-
-    public function isEmulationNeeded(string $code): bool {
+    public function isEmulationNeeded(string $code): bool
+    {
         return strpos(strtolower($code), $this->getKeywordString()) !== false;
     }
 
-    /** @param Token[] $tokens */
-    protected function isKeywordContext(array $tokens, int $pos): bool {
+    protected function isKeywordContext(array $tokens, int $pos): bool
+    {
         $previousNonSpaceToken = $this->getPreviousNonSpaceToken($tokens, $pos);
-        return $previousNonSpaceToken === null || $previousNonSpaceToken->id !== \T_OBJECT_OPERATOR;
+        return $previousNonSpaceToken === null || $previousNonSpaceToken[0] !== \T_OBJECT_OPERATOR;
     }
 
-    public function emulate(string $code, array $tokens): array {
+    public function emulate(string $code, array $tokens): array
+    {
         $keywordString = $this->getKeywordString();
         foreach ($tokens as $i => $token) {
-            if ($token->id === T_STRING && strtolower($token->text) === $keywordString
+            if ($token[0] === T_STRING && strtolower($token[1]) === $keywordString
                     && $this->isKeywordContext($tokens, $i)) {
-                $token->id = $this->getKeywordToken();
+                $tokens[$i][0] = $this->getKeywordToken();
             }
         }
 
         return $tokens;
     }
 
-    /** @param Token[] $tokens */
-    private function getPreviousNonSpaceToken(array $tokens, int $start): ?Token {
+    /**
+     * @param mixed[] $tokens
+     * @return array|string|null
+     */
+    private function getPreviousNonSpaceToken(array $tokens, int $start)
+    {
         for ($i = $start - 1; $i >= 0; --$i) {
-            if ($tokens[$i]->id === T_WHITESPACE) {
+            if ($tokens[$i][0] === T_WHITESPACE) {
                 continue;
             }
 
@@ -43,11 +48,12 @@ abstract class KeywordEmulator extends TokenEmulator {
         return null;
     }
 
-    public function reverseEmulate(string $code, array $tokens): array {
+    public function reverseEmulate(string $code, array $tokens): array
+    {
         $keywordToken = $this->getKeywordToken();
         foreach ($tokens as $i => $token) {
-            if ($token->id === $keywordToken) {
-                $token->id = \T_STRING;
+            if ($token[0] === $keywordToken) {
+                $tokens[$i][0] = \T_STRING;
             }
         }
 

lib/PhpParser/Lexer/TokenEmulator/MatchTokenEmulator.php

@@ -2,18 +2,22 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
+use PhpParser\Lexer\Emulative;
 
-final class MatchTokenEmulator extends KeywordEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 0);
+final class MatchTokenEmulator extends KeywordEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_8_0;
     }
 
-    public function getKeywordString(): string {
+    public function getKeywordString(): string
+    {
         return 'match';
     }
 
-    public function getKeywordToken(): int {
+    public function getKeywordToken(): int
+    {
         return \T_MATCH;
     }
 }

lib/PhpParser/Lexer/TokenEmulator/NullsafeTokenEmulator.php

@@ -2,58 +2,65 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-use PhpParser\Token;
+use PhpParser\Lexer\Emulative;
 
-final class NullsafeTokenEmulator extends TokenEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 0);
+final class NullsafeTokenEmulator extends TokenEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_8_0;
     }
 
-    public function isEmulationNeeded(string $code): bool {
+    public function isEmulationNeeded(string $code): bool
+    {
         return strpos($code, '?->') !== false;
     }
 
-    public function emulate(string $code, array $tokens): array {
+    public function emulate(string $code, array $tokens): array
+    {
         // We need to manually iterate and manage a count because we'll change
         // the tokens array on the way
+        $line = 1;
         for ($i = 0, $c = count($tokens); $i < $c; ++$i) {
-            $token = $tokens[$i];
-            if ($token->text === '?' && isset($tokens[$i + 1]) && $tokens[$i + 1]->id === \T_OBJECT_OPERATOR) {
+            if ($tokens[$i] === '?' && isset($tokens[$i + 1]) && $tokens[$i + 1][0] === \T_OBJECT_OPERATOR) {
                 array_splice($tokens, $i, 2, [
-                    new Token(\T_NULLSAFE_OBJECT_OPERATOR, '?->', $token->line, $token->pos),
+                    [\T_NULLSAFE_OBJECT_OPERATOR, '?->', $line]
                 ]);
                 $c--;
                 continue;
             }
 
             // Handle ?-> inside encapsed string.
-            if ($token->id === \T_ENCAPSED_AND_WHITESPACE && isset($tokens[$i - 1])
-                && $tokens[$i - 1]->id === \T_VARIABLE
-                && preg_match('/^\?->([a-zA-Z_\x80-\xff][a-zA-Z0-9_\x80-\xff]*)/', $token->text, $matches)
+            if ($tokens[$i][0] === \T_ENCAPSED_AND_WHITESPACE && isset($tokens[$i - 1])
+                && $tokens[$i - 1][0] === \T_VARIABLE
+                && preg_match('/^\?->([a-zA-Z_\x80-\xff][a-zA-Z0-9_\x80-\xff]*)/', $tokens[$i][1], $matches)
             ) {
                 $replacement = [
-                    new Token(\T_NULLSAFE_OBJECT_OPERATOR, '?->', $token->line, $token->pos),
-                    new Token(\T_STRING, $matches[1], $token->line, $token->pos + 3),
+                    [\T_NULLSAFE_OBJECT_OPERATOR, '?->', $line],
+                    [\T_STRING, $matches[1], $line],
                 ];
-                $matchLen = \strlen($matches[0]);
-                if ($matchLen !== \strlen($token->text)) {
-                    $replacement[] = new Token(
+                if (\strlen($matches[0]) !== \strlen($tokens[$i][1])) {
+                    $replacement[] = [
                         \T_ENCAPSED_AND_WHITESPACE,
-                        \substr($token->text, $matchLen),
-                        $token->line, $token->pos + $matchLen
-                    );
+                        \substr($tokens[$i][1], \strlen($matches[0])),
+                        $line
+                    ];
                 }
                 array_splice($tokens, $i, 1, $replacement);
                 $c += \count($replacement) - 1;
                 continue;
             }
+
+            if (\is_array($tokens[$i])) {
+                $line += substr_count($tokens[$i][1], "\n");
+            }
         }
 
         return $tokens;
     }
 
-    public function reverseEmulate(string $code, array $tokens): array {
+    public function reverseEmulate(string $code, array $tokens): array
+    {
         // ?-> was not valid code previously, don't bother.
         return $tokens;
     }

lib/PhpParser/Lexer/TokenEmulator/NumericLiteralSeparatorEmulator.php

@@ -2,63 +2,70 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-use PhpParser\Token;
+use PhpParser\Lexer\Emulative;
 
-final class NumericLiteralSeparatorEmulator extends TokenEmulator {
-    private const BIN = '(?:0b[01]+(?:_[01]+)*)';
-    private const HEX = '(?:0x[0-9a-f]+(?:_[0-9a-f]+)*)';
-    private const DEC = '(?:[0-9]+(?:_[0-9]+)*)';
-    private const SIMPLE_FLOAT = '(?:' . self::DEC . '\.' . self::DEC . '?|\.' . self::DEC . ')';
-    private const EXP = '(?:e[+-]?' . self::DEC . ')';
-    private const FLOAT = '(?:' . self::SIMPLE_FLOAT . self::EXP . '?|' . self::DEC . self::EXP . ')';
-    private const NUMBER = '~' . self::FLOAT . '|' . self::BIN . '|' . self::HEX . '|' . self::DEC . '~iA';
+final class NumericLiteralSeparatorEmulator extends TokenEmulator
+{
+    const BIN = '(?:0b[01]+(?:_[01]+)*)';
+    const HEX = '(?:0x[0-9a-f]+(?:_[0-9a-f]+)*)';
+    const DEC = '(?:[0-9]+(?:_[0-9]+)*)';
+    const SIMPLE_FLOAT = '(?:' . self::DEC . '\.' . self::DEC . '?|\.' . self::DEC . ')';
+    const EXP = '(?:e[+-]?' . self::DEC . ')';
+    const FLOAT = '(?:' . self::SIMPLE_FLOAT . self::EXP . '?|' . self::DEC . self::EXP . ')';
+    const NUMBER = '~' . self::FLOAT . '|' . self::BIN . '|' . self::HEX . '|' . self::DEC . '~iA';
 
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(7, 4);
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_7_4;
     }
 
-    public function isEmulationNeeded(string $code): bool {
+    public function isEmulationNeeded(string $code) : bool
+    {
         return preg_match('~[0-9]_[0-9]~', $code)
             || preg_match('~0x[0-9a-f]+_[0-9a-f]~i', $code);
     }
 
-    public function emulate(string $code, array $tokens): array {
+    public function emulate(string $code, array $tokens): array
+    {
         // We need to manually iterate and manage a count because we'll change
         // the tokens array on the way
+        $codeOffset = 0;
         for ($i = 0, $c = count($tokens); $i < $c; ++$i) {
             $token = $tokens[$i];
-            $tokenLen = \strlen($token->text);
+            $tokenLen = \strlen(\is_array($token) ? $token[1] : $token);
 
-            if ($token->id !== \T_LNUMBER && $token->id !== \T_DNUMBER) {
+            if ($token[0] !== T_LNUMBER && $token[0] !== T_DNUMBER) {
+                $codeOffset += $tokenLen;
                 continue;
             }
 
-            $res = preg_match(self::NUMBER, $code, $matches, 0, $token->pos);
+            $res = preg_match(self::NUMBER, $code, $matches, 0, $codeOffset);
             assert($res, "No number at number token position");
 
             $match = $matches[0];
             $matchLen = \strlen($match);
             if ($matchLen === $tokenLen) {
                 // Original token already holds the full number.
+                $codeOffset += $tokenLen;
                 continue;
             }
 
             $tokenKind = $this->resolveIntegerOrFloatToken($match);
-            $newTokens = [new Token($tokenKind, $match, $token->line, $token->pos)];
+            $newTokens = [[$tokenKind, $match, $token[2]]];
 
             $numTokens = 1;
             $len = $tokenLen;
             while ($matchLen > $len) {
                 $nextToken = $tokens[$i + $numTokens];
-                $nextTokenText = $nextToken->text;
+                $nextTokenText = \is_array($nextToken) ? $nextToken[1] : $nextToken;
                 $nextTokenLen = \strlen($nextTokenText);
 
                 $numTokens++;
                 if ($matchLen < $len + $nextTokenLen) {
                     // Split trailing characters into a partial token.
+                    assert(is_array($nextToken), "Partial token should be an array token");
                     $partialText = substr($nextTokenText, $matchLen - $len);
-                    $newTokens[] = new Token($nextToken->id, $partialText, $nextToken->line, $nextToken->pos);
+                    $newTokens[] = [$nextToken[0], $partialText, $nextToken[2]];
                     break;
                 }
 
@@ -67,12 +74,14 @@ final class NumericLiteralSeparatorEmulator extends TokenEmulator {
 
             array_splice($tokens, $i, $numTokens, $newTokens);
             $c -= $numTokens - \count($newTokens);
+            $codeOffset += $matchLen;
         }
 
         return $tokens;
     }
 
-    private function resolveIntegerOrFloatToken(string $str): int {
+    private function resolveIntegerOrFloatToken(string $str): int
+    {
         $str = str_replace('_', '', $str);
 
         if (stripos($str, '0b') === 0) {
@@ -88,7 +97,8 @@ final class NumericLiteralSeparatorEmulator extends TokenEmulator {
         return is_float($num) ? T_DNUMBER : T_LNUMBER;
     }
 
-    public function reverseEmulate(string $code, array $tokens): array {
+    public function reverseEmulate(string $code, array $tokens): array
+    {
         // Numeric separators were not legal code previously, don't bother.
         return $tokens;
     }

lib/PhpParser/Lexer/TokenEmulator/ReadonlyFunctionTokenEmulator.php

@@ -2,7 +2,7 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
+use PhpParser\Lexer\Emulative;
 
 /*
  * In PHP 8.1, "readonly(" was special cased in the lexer in order to support functions with
@@ -20,8 +20,8 @@ class ReadonlyFunctionTokenEmulator extends KeywordEmulator {
         return \T_READONLY;
     }
 
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 2);
+    public function getPhpVersion(): string {
+        return Emulative::PHP_8_2;
     }
 
     public function reverseEmulate(string $code, array $tokens): array {

lib/PhpParser/Lexer/TokenEmulator/ReadonlyTokenEmulator.php

@@ -2,30 +2,35 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
+use PhpParser\Lexer\Emulative;
 
-final class ReadonlyTokenEmulator extends KeywordEmulator {
-    public function getPhpVersion(): PhpVersion {
-        return PhpVersion::fromComponents(8, 1);
+final class ReadonlyTokenEmulator extends KeywordEmulator
+{
+    public function getPhpVersion(): string
+    {
+        return Emulative::PHP_8_1;
     }
 
-    public function getKeywordString(): string {
+    public function getKeywordString(): string
+    {
         return 'readonly';
     }
 
-    public function getKeywordToken(): int {
+    public function getKeywordToken(): int
+    {
         return \T_READONLY;
     }
 
-    protected function isKeywordContext(array $tokens, int $pos): bool {
+    protected function isKeywordContext(array $tokens, int $pos): bool
+    {
         if (!parent::isKeywordContext($tokens, $pos)) {
             return false;
         }
         // Support "function readonly("
         return !(isset($tokens[$pos + 1]) &&
-                 ($tokens[$pos + 1]->text === '(' ||
-                  ($tokens[$pos + 1]->id === \T_WHITESPACE &&
+                 ($tokens[$pos + 1][0] === '(' ||
+                  ($tokens[$pos + 1][0] === \T_WHITESPACE &&
                    isset($tokens[$pos + 2]) &&
-                   $tokens[$pos + 2]->text === '(')));
+                   $tokens[$pos + 2][0] === '(')));
     }
 }

lib/PhpParser/Lexer/TokenEmulator/ReverseEmulator.php

@@ -2,12 +2,11 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-
 /**
  * Reverses emulation direction of the inner emulator.
  */
-final class ReverseEmulator extends TokenEmulator {
+final class ReverseEmulator extends TokenEmulator
+{
     /** @var TokenEmulator Inner emulator */
     private $emulator;
 
@@ -15,7 +14,7 @@ final class ReverseEmulator extends TokenEmulator {
         $this->emulator = $emulator;
     }
 
-    public function getPhpVersion(): PhpVersion {
+    public function getPhpVersion(): string {
         return $this->emulator->getPhpVersion();
     }
 
@@ -34,4 +33,4 @@ final class ReverseEmulator extends TokenEmulator {
     public function preprocessCode(string $code, array &$patches): string {
         return $code;
     }
-}
+}

lib/PhpParser/Lexer/TokenEmulator/TokenEmulator.php

@@ -2,28 +2,23 @@
 
 namespace PhpParser\Lexer\TokenEmulator;
 
-use PhpParser\PhpVersion;
-use PhpParser\Token;
-
 /** @internal */
-abstract class TokenEmulator {
-    abstract public function getPhpVersion(): PhpVersion;
+abstract class TokenEmulator
+{
+    abstract public function getPhpVersion(): string;
 
     abstract public function isEmulationNeeded(string $code): bool;
 
     /**
-     * @param Token[] $tokens Original tokens
-     * @return Token[] Modified Tokens
+     * @return array Modified Tokens
      */
     abstract public function emulate(string $code, array $tokens): array;
 
     /**
-     * @param Token[] $tokens Original tokens
-     * @return Token[] Modified Tokens
+     * @return array Modified Tokens
      */
     abstract public function reverseEmulate(string $code, array $tokens): array;
 
-    /** @param array{int, string, string}[] $patches */
     public function preprocessCode(string $code, array &$patches): string {
         return $code;
     }

lib/PhpParser/Modifiers.php

@@ -1,69 +0,0 @@
-<?php declare(strict_types=1);
-
-namespace PhpParser;
-
-/**
- * Modifiers used (as a bit mask) by various flags subnodes, for example on classes, functions,
- * properties and constants.
- */
-final class Modifiers {
-    public const PUBLIC    =  1;
-    public const PROTECTED =  2;
-    public const PRIVATE   =  4;
-    public const STATIC    =  8;
-    public const ABSTRACT  = 16;
-    public const FINAL     = 32;
-    public const READONLY  = 64;
-
-    public const VISIBILITY_MASK = 1 | 2 | 4;
-
-    /**
-     * @internal
-     */
-    public static function verifyClassModifier(int $a, int $b): void {
-        if ($a & Modifiers::ABSTRACT && $b & Modifiers::ABSTRACT) {
-            throw new Error('Multiple abstract modifiers are not allowed');
-        }
-
-        if ($a & Modifiers::FINAL && $b & Modifiers::FINAL) {
-            throw new Error('Multiple final modifiers are not allowed');
-        }
-
-        if ($a & Modifiers::READONLY && $b & Modifiers::READONLY) {
-            throw new Error('Multiple readonly modifiers are not allowed');
-        }
-
-        if ($a & 48 && $b & 48) {
-            throw new Error('Cannot use the final modifier on an abstract class');
-        }
-    }
-
-    /**
-     * @internal
-     */
-    public static function verifyModifier(int $a, int $b): void {
-        if ($a & Modifiers::VISIBILITY_MASK && $b & Modifiers::VISIBILITY_MASK) {
-            throw new Error('Multiple access type modifiers are not allowed');
-        }
-
-        if ($a & Modifiers::ABSTRACT && $b & Modifiers::ABSTRACT) {
-            throw new Error('Multiple abstract modifiers are not allowed');
-        }
-
-        if ($a & Modifiers::STATIC && $b & Modifiers::STATIC) {
-            throw new Error('Multiple static modifiers are not allowed');
-        }
-
-        if ($a & Modifiers::FINAL && $b & Modifiers::FINAL) {
-            throw new Error('Multiple final modifiers are not allowed');
-        }
-
-        if ($a & Modifiers::READONLY && $b & Modifiers::READONLY) {
-            throw new Error('Multiple readonly modifiers are not allowed');
-        }
-
-        if ($a & 48 && $b & 48) {
-            throw new Error('Cannot use the final modifier on an abstract class member');
-        }
-    }
-}

lib/PhpParser/NameContext.php

@@ -6,7 +6,8 @@ use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PhpParser\Node\Stmt;
 
-class NameContext {
+class NameContext
+{
     /** @var null|Name Current namespace */
     protected $namespace;
 
@@ -35,7 +36,7 @@ class NameContext {
      *
      * @param Name|null $namespace Null is the global namespace
      */
-    public function startNamespace(?Name $namespace = null): void {
+    public function startNamespace(Name $namespace = null) {
         $this->namespace = $namespace;
         $this->origAliases = $this->aliases = [
             Stmt\Use_::TYPE_NORMAL   => [],
@@ -50,9 +51,9 @@ class NameContext {
      * @param Name   $name        Original name
      * @param string $aliasName   Aliased name
      * @param int    $type        One of Stmt\Use_::TYPE_*
-     * @param array<string, mixed> $errorAttrs Attributes to use to report an error
+     * @param array  $errorAttrs Attributes to use to report an error
      */
-    public function addAlias(Name $name, string $aliasName, int $type, array $errorAttrs = []): void {
+    public function addAlias(Name $name, string $aliasName, int $type, array $errorAttrs = []) {
         // Constant names are case sensitive, everything else case insensitive
         if ($type === Stmt\Use_::TYPE_CONSTANT) {
             $aliasLookupName = $aliasName;
@@ -86,7 +87,7 @@ class NameContext {
      *
      * @return null|Name Namespace (or null if global namespace)
      */
-    public function getNamespace(): ?Name {
+    public function getNamespace() {
         return $this->namespace;
     }
 
@@ -98,7 +99,7 @@ class NameContext {
      *
      * @return null|Name Resolved name, or null if static resolution is not possible
      */
-    public function getResolvedName(Name $name, int $type): ?Name {
+    public function getResolvedName(Name $name, int $type) {
         // don't resolve special class names
         if ($type === Stmt\Use_::TYPE_NORMAL && $name->isSpecialClassName()) {
             if (!$name->isUnqualified()) {
@@ -141,7 +142,7 @@ class NameContext {
      *
      * @return Name Resolved name
      */
-    public function getResolvedClassName(Name $name): Name {
+    public function getResolvedClassName(Name $name) : Name {
         return $this->getResolvedName($name, Stmt\Use_::TYPE_NORMAL);
     }
 
@@ -153,7 +154,7 @@ class NameContext {
      *
      * @return Name[] Possible representations of the name
      */
-    public function getPossibleNames(string $name, int $type): array {
+    public function getPossibleNames(string $name, int $type) : array {
         $lcName = strtolower($name);
 
         if ($type === Stmt\Use_::TYPE_NORMAL) {
@@ -209,7 +210,7 @@ class NameContext {
      *
      * @return Name Shortest representation
      */
-    public function getShortName(string $name, int $type): Name {
+    public function getShortName(string $name, int $type) : Name {
         $possibleNames = $this->getPossibleNames($name, $type);
 
         // Find shortest name
@@ -223,10 +224,10 @@ class NameContext {
             }
         }
 
-        return $shortestName;
+       return $shortestName;
     }
 
-    private function resolveAlias(Name $name, int $type): ?FullyQualified {
+    private function resolveAlias(Name $name, $type) {
         $firstPart = $name->getFirst();
 
         if ($name->isQualified()) {
@@ -249,7 +250,7 @@ class NameContext {
         return null;
     }
 
-    private function getNamespaceRelativeName(string $name, string $lcName, int $type): ?Name {
+    private function getNamespaceRelativeName(string $name, string $lcName, int $type) {
         if (null === $this->namespace) {
             return new Name($name);
         }
@@ -270,7 +271,7 @@ class NameContext {
         return null;
     }
 
-    private function normalizeConstName(string $name): string {
+    private function normalizeConstName(string $name) {
         $nsSep = strrpos($name, '\\');
         if (false === $nsSep) {
             return $name;

lib/PhpParser/Node.php

@@ -2,27 +2,28 @@
 
 namespace PhpParser;
 
-interface Node {
+interface Node
+{
     /**
      * Gets the type of the node.
      *
      * @return string Type of the node
      */
-    public function getType(): string;
+    public function getType() : string;
 
     /**
      * Gets the names of the sub nodes.
      *
-     * @return string[] Names of sub nodes
+     * @return array Names of sub nodes
      */
-    public function getSubNodeNames(): array;
+    public function getSubNodeNames() : array;
 
     /**
      * Gets line the node started in (alias of getStartLine).
      *
      * @return int Start line (or -1 if not available)
      */
-    public function getLine(): int;
+    public function getLine() : int;
 
     /**
      * Gets line the node started in.
@@ -31,7 +32,7 @@ interface Node {
      *
      * @return int Start line (or -1 if not available)
      */
-    public function getStartLine(): int;
+    public function getStartLine() : int;
 
     /**
      * Gets the line the node ended in.
@@ -40,7 +41,7 @@ interface Node {
      *
      * @return int End line (or -1 if not available)
      */
-    public function getEndLine(): int;
+    public function getEndLine() : int;
 
     /**
      * Gets the token offset of the first token that is part of this node.
@@ -51,7 +52,7 @@ interface Node {
      *
      * @return int Token start position (or -1 if not available)
      */
-    public function getStartTokenPos(): int;
+    public function getStartTokenPos() : int;
 
     /**
      * Gets the token offset of the last token that is part of this node.
@@ -62,7 +63,7 @@ interface Node {
      *
      * @return int Token end position (or -1 if not available)
      */
-    public function getEndTokenPos(): int;
+    public function getEndTokenPos() : int;
 
     /**
      * Gets the file offset of the first character that is part of this node.
@@ -71,7 +72,7 @@ interface Node {
      *
      * @return int File start position (or -1 if not available)
      */
-    public function getStartFilePos(): int;
+    public function getStartFilePos() : int;
 
     /**
      * Gets the file offset of the last character that is part of this node.
@@ -80,7 +81,7 @@ interface Node {
      *
      * @return int File end position (or -1 if not available)
      */
-    public function getEndFilePos(): int;
+    public function getEndFilePos() : int;
 
     /**
      * Gets all comments directly preceding this node.
@@ -89,14 +90,14 @@ interface Node {
      *
      * @return Comment[]
      */
-    public function getComments(): array;
+    public function getComments() : array;
 
     /**
      * Gets the doc comment of the node.
      *
      * @return null|Comment\Doc Doc comment object or null
      */
-    public function getDocComment(): ?Comment\Doc;
+    public function getDocComment();
 
     /**
      * Sets the doc comment of the node.
@@ -105,7 +106,7 @@ interface Node {
      *
      * @param Comment\Doc $docComment Doc comment to set
      */
-    public function setDocComment(Comment\Doc $docComment): void;
+    public function setDocComment(Comment\Doc $docComment);
 
     /**
      * Sets an attribute on a node.
@@ -113,7 +114,7 @@ interface Node {
      * @param string $key
      * @param mixed  $value
      */
-    public function setAttribute(string $key, $value): void;
+    public function setAttribute(string $key, $value);
 
     /**
      * Returns whether an attribute exists.
@@ -122,7 +123,7 @@ interface Node {
      *
      * @return bool
      */
-    public function hasAttribute(string $key): bool;
+    public function hasAttribute(string $key) : bool;
 
     /**
      * Returns the value of an attribute.
@@ -137,14 +138,14 @@ interface Node {
     /**
      * Returns all the attributes of this node.
      *
-     * @return array<string, mixed>
+     * @return array
      */
-    public function getAttributes(): array;
+    public function getAttributes() : array;
 
     /**
      * Replaces all the attributes of this node.
      *
-     * @param array<string, mixed> $attributes
+     * @param array $attributes
      */
-    public function setAttributes(array $attributes): void;
+    public function setAttributes(array $attributes);
 }

lib/PhpParser/Node/Arg.php

@@ -2,9 +2,11 @@
 
 namespace PhpParser\Node;
 
+use PhpParser\Node\VariadicPlaceholder;
 use PhpParser\NodeAbstract;
 
-class Arg extends NodeAbstract {
+class Arg extends NodeAbstract
+{
     /** @var Identifier|null Parameter name (for named parameters) */
     public $name;
     /** @var Expr Value to pass */
@@ -20,12 +22,12 @@ class Arg extends NodeAbstract {
      * @param Expr  $value      Value to pass
      * @param bool  $byRef      Whether to pass by ref
      * @param bool  $unpack     Whether to unpack the argument
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param array $attributes Additional attributes
      * @param Identifier|null $name Parameter name (for named parameters)
      */
     public function __construct(
         Expr $value, bool $byRef = false, bool $unpack = false, array $attributes = [],
-        ?Identifier $name = null
+        Identifier $name = null
     ) {
         $this->attributes = $attributes;
         $this->name = $name;
@@ -34,11 +36,11 @@ class Arg extends NodeAbstract {
         $this->unpack = $unpack;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['name', 'value', 'byRef', 'unpack'];
     }
-
-    public function getType(): string {
+    
+    public function getType() : string {
         return 'Arg';
     }
 }

lib/PhpParser/Node/ArrayItem.php

@@ -1,43 +0,0 @@
-<?php declare(strict_types=1);
-
-namespace PhpParser\Node;
-
-use PhpParser\NodeAbstract;
-
-class ArrayItem extends NodeAbstract {
-    /** @var null|Expr Key */
-    public $key;
-    /** @var Expr Value */
-    public $value;
-    /** @var bool Whether to assign by reference */
-    public $byRef;
-    /** @var bool Whether to unpack the argument */
-    public $unpack;
-
-    /**
-     * Constructs an array item node.
-     *
-     * @param Expr      $value      Value
-     * @param null|Expr $key        Key
-     * @param bool      $byRef      Whether to assign by reference
-     * @param array<string, mixed> $attributes Additional attributes
-     */
-    public function __construct(Expr $value, ?Expr $key = null, bool $byRef = false, array $attributes = [], bool $unpack = false) {
-        $this->attributes = $attributes;
-        $this->key = $key;
-        $this->value = $value;
-        $this->byRef = $byRef;
-        $this->unpack = $unpack;
-    }
-
-    public function getSubNodeNames(): array {
-        return ['key', 'value', 'byRef', 'unpack'];
-    }
-
-    public function getType(): string {
-        return 'ArrayItem';
-    }
-}
-
-// @deprecated compatibility alias
-class_alias(ArrayItem::class, Expr\ArrayItem::class);

lib/PhpParser/Node/Attribute.php

@@ -5,17 +5,18 @@ namespace PhpParser\Node;
 use PhpParser\Node;
 use PhpParser\NodeAbstract;
 
-class Attribute extends NodeAbstract {
+class Attribute extends NodeAbstract
+{
     /** @var Name Attribute name */
     public $name;
 
-    /** @var list<Arg> Attribute arguments */
+    /** @var Arg[] Attribute arguments */
     public $args;
 
     /**
      * @param Node\Name $name       Attribute name
-     * @param list<Arg> $args Attribute arguments
-     * @param array<string, mixed> $attributes Additional node attributes
+     * @param Arg[]     $args       Attribute arguments
+     * @param array     $attributes Additional node attributes
      */
     public function __construct(Name $name, array $args = [], array $attributes = []) {
         $this->attributes = $attributes;
@@ -23,11 +24,11 @@ class Attribute extends NodeAbstract {
         $this->args = $args;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['name', 'args'];
     }
 
-    public function getType(): string {
+    public function getType() : string {
         return 'Attribute';
     }
 }

lib/PhpParser/Node/AttributeGroup.php

@@ -2,26 +2,28 @@
 
 namespace PhpParser\Node;
 
+use PhpParser\Node;
 use PhpParser\NodeAbstract;
 
-class AttributeGroup extends NodeAbstract {
+class AttributeGroup extends NodeAbstract
+{
     /** @var Attribute[] Attributes */
     public $attrs;
 
     /**
      * @param Attribute[] $attrs PHP attributes
-     * @param array<string, mixed> $attributes Additional node attributes
+     * @param array $attributes Additional node attributes
      */
     public function __construct(array $attrs, array $attributes = []) {
         $this->attributes = $attributes;
         $this->attrs = $attrs;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['attrs'];
     }
 
-    public function getType(): string {
+    public function getType() : string {
         return 'AttributeGroup';
     }
 }

lib/PhpParser/Node/ClosureUse.php

@@ -1,36 +0,0 @@
-<?php declare(strict_types=1);
-
-namespace PhpParser\Node;
-
-use PhpParser\NodeAbstract;
-
-class ClosureUse extends NodeAbstract {
-    /** @var Expr\Variable Variable to use */
-    public $var;
-    /** @var bool Whether to use by reference */
-    public $byRef;
-
-    /**
-     * Constructs a closure use node.
-     *
-     * @param Expr\Variable $var        Variable to use
-     * @param bool          $byRef      Whether to use by reference
-     * @param array<string, mixed> $attributes Additional attributes
-     */
-    public function __construct(Expr\Variable $var, bool $byRef = false, array $attributes = []) {
-        $this->attributes = $attributes;
-        $this->var = $var;
-        $this->byRef = $byRef;
-    }
-
-    public function getSubNodeNames(): array {
-        return ['var', 'byRef'];
-    }
-
-    public function getType(): string {
-        return 'ClosureUse';
-    }
-}
-
-// @deprecated compatibility alias
-class_alias(ClosureUse::class, Expr\ClosureUse::class);

lib/PhpParser/Node/ComplexType.php

@@ -9,5 +9,6 @@ use PhpParser\NodeAbstract;
  *
  * It does not provide any shared behavior and exists only for type-checking purposes.
  */
-abstract class ComplexType extends NodeAbstract {
+abstract class ComplexType extends NodeAbstract
+{
 }

lib/PhpParser/Node/Const_.php

@@ -4,7 +4,8 @@ namespace PhpParser\Node;
 
 use PhpParser\NodeAbstract;
 
-class Const_ extends NodeAbstract {
+class Const_ extends NodeAbstract
+{
     /** @var Identifier Name */
     public $name;
     /** @var Expr Value */
@@ -18,7 +19,7 @@ class Const_ extends NodeAbstract {
      *
      * @param string|Identifier $name       Name
      * @param Expr              $value      Value
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param array             $attributes Additional attributes
      */
     public function __construct($name, Expr $value, array $attributes = []) {
         $this->attributes = $attributes;
@@ -26,11 +27,11 @@ class Const_ extends NodeAbstract {
         $this->value = $value;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['name', 'value'];
     }
 
-    public function getType(): string {
+    public function getType() : string {
         return 'Const';
     }
 }

lib/PhpParser/Node/DeclareItem.php

@@ -1,37 +0,0 @@
-<?php declare(strict_types=1);
-
-namespace PhpParser\Node;
-
-use PhpParser\Node;
-use PhpParser\NodeAbstract;
-
-class DeclareItem extends NodeAbstract {
-    /** @var Node\Identifier Key */
-    public $key;
-    /** @var Node\Expr Value */
-    public $value;
-
-    /**
-     * Constructs a declare key=>value pair node.
-     *
-     * @param string|Node\Identifier $key        Key
-     * @param Node\Expr              $value      Value
-     * @param array<string, mixed> $attributes Additional attributes
-     */
-    public function __construct($key, Node\Expr $value, array $attributes = []) {
-        $this->attributes = $attributes;
-        $this->key = \is_string($key) ? new Node\Identifier($key) : $key;
-        $this->value = $value;
-    }
-
-    public function getSubNodeNames(): array {
-        return ['key', 'value'];
-    }
-
-    public function getType(): string {
-        return 'DeclareItem';
-    }
-}
-
-// @deprecated compatibility alias
-class_alias(DeclareItem::class, Stmt\DeclareDeclare::class);

lib/PhpParser/Node/Expr.php

@@ -4,5 +4,6 @@ namespace PhpParser\Node;
 
 use PhpParser\NodeAbstract;
 
-abstract class Expr extends NodeAbstract {
+abstract class Expr extends NodeAbstract
+{
 }

lib/PhpParser/Node/Expr/ArrayDimFetch.php

@@ -4,7 +4,8 @@ namespace PhpParser\Node\Expr;
 
 use PhpParser\Node\Expr;
 
-class ArrayDimFetch extends Expr {
+class ArrayDimFetch extends Expr
+{
     /** @var Expr Variable */
     public $var;
     /** @var null|Expr Array index / dim */
@@ -15,19 +16,19 @@ class ArrayDimFetch extends Expr {
      *
      * @param Expr      $var        Variable
      * @param null|Expr $dim        Array index / dim
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param array     $attributes Additional attributes
      */
-    public function __construct(Expr $var, ?Expr $dim = null, array $attributes = []) {
+    public function __construct(Expr $var, Expr $dim = null, array $attributes = []) {
         $this->attributes = $attributes;
         $this->var = $var;
         $this->dim = $dim;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['var', 'dim'];
     }
-
-    public function getType(): string {
+    
+    public function getType() : string {
         return 'Expr_ArrayDimFetch';
     }
 }

lib/PhpParser/Node/Expr/ArrayItem.php

@@ -1,3 +1,41 @@
 <?php declare(strict_types=1);
 
-require __DIR__ . '/../ArrayItem.php';
+namespace PhpParser\Node\Expr;
+
+use PhpParser\Node\Expr;
+
+class ArrayItem extends Expr
+{
+    /** @var null|Expr Key */
+    public $key;
+    /** @var Expr Value */
+    public $value;
+    /** @var bool Whether to assign by reference */
+    public $byRef;
+    /** @var bool Whether to unpack the argument */
+    public $unpack;
+
+    /**
+     * Constructs an array item node.
+     *
+     * @param Expr      $value      Value
+     * @param null|Expr $key        Key
+     * @param bool      $byRef      Whether to assign by reference
+     * @param array     $attributes Additional attributes
+     */
+    public function __construct(Expr $value, Expr $key = null, bool $byRef = false, array $attributes = [], bool $unpack = false) {
+        $this->attributes = $attributes;
+        $this->key = $key;
+        $this->value = $value;
+        $this->byRef = $byRef;
+        $this->unpack = $unpack;
+    }
+
+    public function getSubNodeNames() : array {
+        return ['key', 'value', 'byRef', 'unpack'];
+    }
+
+    public function getType() : string {
+        return 'Expr_ArrayItem';
+    }
+}

lib/PhpParser/Node/Expr/Array_.php

@@ -2,33 +2,33 @@
 
 namespace PhpParser\Node\Expr;
 
-use PhpParser\Node\ArrayItem;
 use PhpParser\Node\Expr;
 
-class Array_ extends Expr {
+class Array_ extends Expr
+{
     // For use in "kind" attribute
-    public const KIND_LONG = 1;  // array() syntax
-    public const KIND_SHORT = 2; // [] syntax
+    const KIND_LONG = 1;  // array() syntax
+    const KIND_SHORT = 2; // [] syntax
 
-    /** @var ArrayItem[] Items */
+    /** @var (ArrayItem|null)[] Items */
     public $items;
 
     /**
      * Constructs an array node.
      *
-     * @param ArrayItem[] $items      Items of the array
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param (ArrayItem|null)[] $items      Items of the array
+     * @param array       $attributes Additional attributes
      */
     public function __construct(array $items = [], array $attributes = []) {
         $this->attributes = $attributes;
         $this->items = $items;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['items'];
     }
-
-    public function getType(): string {
+    
+    public function getType() : string {
         return 'Expr_Array';
     }
 }

lib/PhpParser/Node/Expr/ArrowFunction.php

@@ -6,7 +6,8 @@ use PhpParser\Node;
 use PhpParser\Node\Expr;
 use PhpParser\Node\FunctionLike;
 
-class ArrowFunction extends Expr implements FunctionLike {
+class ArrowFunction extends Expr implements FunctionLike
+{
     /** @var bool */
     public $static;
 
@@ -25,23 +26,16 @@ class ArrowFunction extends Expr implements FunctionLike {
     public $attrGroups;
 
     /**
-     * @param array{
-     *     expr: Expr,
-     *     static?: bool,
-     *     byRef?: bool,
-     *     params?: Node\Param[],
-     *     returnType?: null|string|Node\Identifier|Node\Name|Node\ComplexType,
-     *     attrGroups?: Node\AttributeGroup[]
-     * } $subNodes Array of the following subnodes:
-     *             'expr'                  : Expression body
-     *             'static'     => false   : Whether the closure is static
-     *             'byRef'      => false   : Whether to return by reference
-     *             'params'     => array() : Parameters
-     *             'returnType' => null    : Return type
-     *             'attrGroups' => array() : PHP attribute groups
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param array $subNodes   Array of the following optional subnodes:
+     *                          'static'     => false   : Whether the closure is static
+     *                          'byRef'      => false   : Whether to return by reference
+     *                          'params'     => array() : Parameters
+     *                          'returnType' => null    : Return type
+     *                          'expr'       => Expr    : Expression body
+     *                          'attrGroups' => array() : PHP attribute groups
+     * @param array $attributes Additional attributes
      */
-    public function __construct(array $subNodes, array $attributes = []) {
+    public function __construct(array $subNodes = [], array $attributes = []) {
         $this->attributes = $attributes;
         $this->static = $subNodes['static'] ?? false;
         $this->byRef = $subNodes['byRef'] ?? false;
@@ -52,15 +46,15 @@ class ArrowFunction extends Expr implements FunctionLike {
         $this->attrGroups = $subNodes['attrGroups'] ?? [];
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['attrGroups', 'static', 'byRef', 'params', 'returnType', 'expr'];
     }
 
-    public function returnsByRef(): bool {
+    public function returnsByRef() : bool {
         return $this->byRef;
     }
 
-    public function getParams(): array {
+    public function getParams() : array {
         return $this->params;
     }
 
@@ -68,18 +62,18 @@ class ArrowFunction extends Expr implements FunctionLike {
         return $this->returnType;
     }
 
-    public function getAttrGroups(): array {
+    public function getAttrGroups() : array {
         return $this->attrGroups;
     }
 
     /**
      * @return Node\Stmt\Return_[]
      */
-    public function getStmts(): array {
+    public function getStmts() : array {
         return [new Node\Stmt\Return_($this->expr)];
     }
 
-    public function getType(): string {
+    public function getType() : string {
         return 'Expr_ArrowFunction';
     }
 }

lib/PhpParser/Node/Expr/Assign.php

@@ -4,7 +4,8 @@ namespace PhpParser\Node\Expr;
 
 use PhpParser\Node\Expr;
 
-class Assign extends Expr {
+class Assign extends Expr
+{
     /** @var Expr Variable */
     public $var;
     /** @var Expr Expression */
@@ -15,7 +16,7 @@ class Assign extends Expr {
      *
      * @param Expr  $var        Variable
      * @param Expr  $expr       Expression
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param array $attributes Additional attributes
      */
     public function __construct(Expr $var, Expr $expr, array $attributes = []) {
         $this->attributes = $attributes;
@@ -23,11 +24,11 @@ class Assign extends Expr {
         $this->expr = $expr;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['var', 'expr'];
     }
-
-    public function getType(): string {
+    
+    public function getType() : string {
         return 'Expr_Assign';
     }
 }

lib/PhpParser/Node/Expr/AssignOp.php

@@ -4,7 +4,8 @@ namespace PhpParser\Node\Expr;
 
 use PhpParser\Node\Expr;
 
-abstract class AssignOp extends Expr {
+abstract class AssignOp extends Expr
+{
     /** @var Expr Variable */
     public $var;
     /** @var Expr Expression */
@@ -15,7 +16,7 @@ abstract class AssignOp extends Expr {
      *
      * @param Expr  $var        Variable
      * @param Expr  $expr       Expression
-     * @param array<string, mixed> $attributes Additional attributes
+     * @param array $attributes Additional attributes
      */
     public function __construct(Expr $var, Expr $expr, array $attributes = []) {
         $this->attributes = $attributes;
@@ -23,7 +24,7 @@ abstract class AssignOp extends Expr {
         $this->expr = $expr;
     }
 
-    public function getSubNodeNames(): array {
+    public function getSubNodeNames() : array {
         return ['var', 'expr'];
     }
 }

lib/PhpParser/Node/Expr/AssignOp/BitwiseAnd.php

@@ -4,8 +4,9 @@ namespace PhpParser\Node\Expr\AssignOp;
 
 use PhpParser\Node\Expr\AssignOp;
 
-class BitwiseAnd extends AssignOp {
-    public function getType(): string {
+class BitwiseAnd extends AssignOp
+{
+    public function getType() : string {
         return 'Expr_AssignOp_BitwiseAnd';
     }
 }

lib/PhpParser/Node/Expr/AssignOp/BitwiseOr.php

@@ -4,8 +4,9 @@ namespace PhpParser\Node\Expr\AssignOp;
 
 use PhpParser\Node\Expr\AssignOp;
 
-class BitwiseOr extends AssignOp {
-    public function getType(): string {
+class BitwiseOr extends AssignOp
+{
+    public function getType() : string {
         return 'Expr_AssignOp_BitwiseOr';
     }
 }

lib/PhpParser/Node/Expr/AssignOp/BitwiseXor.php

@@ -4,8 +4,9 @@ namespace PhpParser\Node\Expr\AssignOp;
 
 use PhpParser\Node\Expr\AssignOp;
 
-class BitwiseXor extends AssignOp {
-    public function getType(): string {
+class BitwiseXor extends AssignOp
+{
+    public function getType() : string {
         return 'Expr_AssignOp_BitwiseXor';
     }
 }

lib/PhpParser/Node/Expr/AssignOp/Coalesce.php

@@ -4,8 +4,9 @@ namespace PhpParser\Node\Expr\AssignOp;
 
 use PhpParser\Node\Expr\AssignOp;
 
-class Coalesce extends AssignOp {
-    public function getType(): string {
+class Coalesce extends AssignOp
+{
+    public function getType() : string {
         return 'Expr_AssignOp_Coalesce';
     }
 }