2
78%
master: 79%

Ran 03 Apr 2024 02:06AM UTC

Files 259

Run time 5s

Badge

Embed ▾

Committed 03 Apr 2024 02:02AM UTC coverage: 70.848% (+0.003%) from 70.845%

Job # 8532137691.2

Build Type

push

github

Committed by

jrfnl

Commit Message

File::getMethodProperties(): skip over closure use statements

This PR improves performance of the `File::getMethodProperties()` method and prevents incorrect return type information for closures `use` clauses containing invalid variable imports in the `use` clause (defensive coding).

Closure `use` statements can only import plain variables, not properties or other more complex variables.

As things were, when such "illegal" variables were imported in a closure `use`, the information for the return type could get mangled.
While this would be a parse error, for the purposes of static analysis, the `File::getMethodProperties()` method should still handle this correctly.

This commit updates the `File::getMethodProperties()` method to always skip over the complete `use` clause, which prevents the issue and improves performance as the same time (less token walking).

Includes unit tests.

Run Details

16903 of 23858 relevant lines covered (70.85%)

18.45 hits per line

PHPCSStandards / PHP_CodeSniffer / 8532137691 / 2
78%
master: 79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job php-8.1-custom-ini-true - 8532137691.2

PHPCSStandards / PHP_CodeSniffer / 8532137691 / 2 78% master: 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job php-8.1-custom-ini-true - 8532137691.2

PHPCSStandards / PHP_CodeSniffer / 8532137691 / 2
78%
master: 79%

README BADGES
x