PSR2/PSR12/ControlStructureSpacing: don't listen for `T_ELSE`

jrfnl commented 1 year ago

Description

No functional changes.

These sniffs only concern themselves with control structures which take parentheses, so listening for T_ELSE is unnecessary as the sniffs will never do anything for that token (they each bow out on no 'parenthesis_opener'/'parenthesis_closer' found).

Suggested changelog entry

N/A (no functional changes)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] This change is only breaking for integrators, not for external standards or end-users.
[ ] Documentation improvement

DannyvdSluijs commented 1 year ago

Since the changes have no effect (as it is a reduction of the tokens it wants to process) the verification is done by confirming the description of this PR is correct and secondly that the code changes are as expected.

jrfnl commented 9 months ago

Closing as replaced by https://github.com/PHPCSStandards/PHP_CodeSniffer/pull/83

squizlabs / PHP_CodeSniffer