Skip to content

Commit fad249a

Browse files
committed
Squiz.Formatting.OperatorBracket reporting false positive for exit statements (ref #3099)
1 parent 2f844c3 commit fad249a

File tree

3 files changed

+5
-0
lines changed

3 files changed

+5
-0
lines changed

src/Standards/Squiz/Sniffs/Formatting/OperatorBracketSniff.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,7 @@ public function process(File $phpcsFile, $stackPtr)
107107
T_OPEN_CURLY_BRACKET => true,
108108
T_OPEN_SHORT_ARRAY => true,
109109
T_CASE => true,
110+
T_EXIT => true,
110111
];
111112

112113
if (isset($invalidTokens[$tokens[$previousToken]['code']]) === true) {

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,3 +172,5 @@ $test = (1 * static::TEST);
172172
$test = myfunc(1 * static::TEST);
173173

174174
$errorPos = $params[$x]?->getLine() + $commentStart;
175+
176+
exit -1;

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc.fixed

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,3 +172,5 @@ $test = (1 * static::TEST);
172172
$test = myfunc(1 * static::TEST);
173173

174174
$errorPos = ($params[$x]?->getLine() + $commentStart);
175+
176+
exit -1;

0 commit comments

Comments
 (0)