MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-01-08 15:09:39 +07:00
Code Issues Packages Projects Releases Wiki Activity

[java-intentions] EA-465575 - AE: JavaParserUtil.parseFragment

Browse Source 
GitOrigin-RevId: 51d05798ab9a127508bcf531a3aa1a13cdda5aae
This commit is contained in:
Tagir Valeev
2022-08-01 10:36:10 +02:00
committed by intellij-monorepo-bot
parent b5c5a86f77
commit 46c6cb94ad
6 changed files with 32 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/ConvertSwitchToIfIntention.java
View File

@@ -310,7 +310,7 @@ public class ConvertSwitchToIfIntention implements IntentionActionWithFixAllOpti
if (caseElement instanceof PsiExpression) {
PsiExpression caseExpression = PsiUtil.skipParenthesizedExprDown((PsiExpression)caseElement);
String caseValue = getCaseValueText(caseExpression, commentTracker);
if (useEquals && !ExpressionUtils.isNullLiteral(caseExpression)) {
if (useEquals && caseExpression != null && !(caseExpression.getType() instanceof PsiPrimitiveType)) {
if (PsiPrecedenceUtil.getPrecedence(caseExpression) > PsiPrecedenceUtil.METHOD_CALL_PRECEDENCE) {
caseValue = "(" + caseValue + ")";
}

7
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/convertSwitchToIf/afterConstantOnObject.java Normal file
View File

@@ -0,0 +1,7 @@
// "Replace 'switch' with 'if'" "true-preview"
class Test {
void test(Object obj) {
if (obj == 1) {
}
}
}

2
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/convertSwitchToIf/afterIncorrectPattern.java
View File

@@ -1,7 +1,7 @@
// "Replace 'switch' with 'if'" "true-preview"
class Test {
void test(Object obj) {
if ((obj instanceof String s).equals(obj)) {
if (obj == obj instanceof String s) {
}
}
}

7
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/convertSwitchToIf/afterInstanceOfOnObject.java Normal file
View File

@@ -0,0 +1,7 @@
// "Replace 'switch' with 'if'" "true-preview"
class Test {
void test(Object obj) {
if (obj == obj instanceof String) {
}
}
}

8
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/convertSwitchToIf/beforeConstantOnObject.java Normal file
View File

@@ -0,0 +1,8 @@
// "Replace 'switch' with 'if'" "true-preview"
class Test {
void test(Object obj) {
<caret>switch (obj) {
case 1 -> {}
}
}
}

8
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/convertSwitchToIf/beforeInstanceOfOnObject.java Normal file
View File

@@ -0,0 +1,8 @@
// "Replace 'switch' with 'if'" "true-preview"
class Test {
void test(Object obj) {
<caret>switch (obj) {
case obj instanceof String -> {}
}
}
}