diff --git a/plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ParenthesesUtils.java b/plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ParenthesesUtils.java index 67b5f539a39d..b4d97f52687d 100644 --- a/plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ParenthesesUtils.java +++ b/plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ParenthesesUtils.java @@ -455,8 +455,7 @@ public class ParenthesesUtils { return true; } } - return false; } - return parentPrecedence != childPrecedence; + return parentPrecedence < childPrecedence; } } diff --git a/plugins/IntentionPowerPak/test/com/siyeh/ipp/parentheses/SimplePrecedence.java b/plugins/IntentionPowerPak/test/com/siyeh/ipp/parentheses/SimplePrecedence.java new file mode 100644 index 000000000000..c2bfc6529b8a --- /dev/null +++ b/plugins/IntentionPowerPak/test/com/siyeh/ipp/parentheses/SimplePrecedence.java @@ -0,0 +1,11 @@ +package com.siyeh.ipp.parentheses; +class Inversion { + public void context(boolean a, boolean b, boolean c) { + if ((!a || !b) && !c) { + System.out.println(1); + } + else { + System.out.println(0); + } + } +} \ No newline at end of file diff --git a/plugins/IntentionPowerPak/testSrc/com/siyeh/ipp/parenthesis/UnnecessaryParenthesesIntentionTest.java b/plugins/IntentionPowerPak/testSrc/com/siyeh/ipp/parenthesis/UnnecessaryParenthesesIntentionTest.java index 77bc9e5abede..720442fc883d 100644 --- a/plugins/IntentionPowerPak/testSrc/com/siyeh/ipp/parenthesis/UnnecessaryParenthesesIntentionTest.java +++ b/plugins/IntentionPowerPak/testSrc/com/siyeh/ipp/parenthesis/UnnecessaryParenthesesIntentionTest.java @@ -17,6 +17,7 @@ public class UnnecessaryParenthesesIntentionTest extends IPPTestCase { public void testNotCommutative2() { doTest(); } public void testArrayInitializer() { doTest(); } public void testArrayAccessExpression() { doTest(); } + public void testSimplePrecedence() { assertIntentionNotAvailable(); } @Override protected String getRelativePath() {