SEO_SUBOPTIMAL_EXPRESSION_ORDER false-negative

[boris-petrov]

I have something like:

boolean condition = someComplexExpression();

if (someOtherComplexExpression(1, 2) || condition) {
}

But it doesn't trigger the SEO_SUBOPTIMAL_EXPRESSION_ORDER warning. If you can't reproduce it with a simple example, I'l try to work out a working reproduction.

[mebigfatguy]

the problem with or's is that the individual branches have different targets which makes it hard to figure out whether the two are part of the same if condition or not.