Phpinspectionsea: Unnecessary parenthesis completely changes meaning if removed

[Bilge]

Given the following code:

if ((yield $results->advance()) !== false) {

PHPEA claims the inner braces are redundant and thus changes the code to the following:

if (yield $results->advance() !== false) {

The former yields whatever $results->advance() returns, the latter yields a boolean. Since these two statements are non-equivalent this must be considered a bug in the inspection.

[kalessil]

Good catch @Bilge , thank you

[kalessil]

Fixed