PHPCSStandards / PHP_CodeSniffer / 12048293772

78%

master: 79%

push

github

Generic/OpeningFunctionBraceKernighanRitchie: simplify logic to find end of the function signature

When finding the end of the function signature, the sniff had an
unnecessary block of code to change the `$end` parameter passed to
`findPrevious()` when handling a closure with a `use` statement.

Since we are looking for the first non-empty token before the opening
curly brace, it is not necessary to change the value of the `$end`
parameter. Actually, we don't even need to limit the search and we can
pass `null` (the default value). The returned first non-empty token will
be the same anyway regardless if `$end` is the closing parenthesis
before `use`, after `use` or `null`.

1 of 1 new or added line in 1 file covered. (100.0%)

18753 of 24580 relevant lines covered (76.29%)

75.34 hits per line