MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-09-27 10:03:11 +07:00
Code Issues Packages Projects Releases Wiki Activity

IDEA-83207 False "Duplicate switch case" warning in Groovy

Browse Source
This commit is contained in:
Maxim.Medvedev
2012-03-21 16:58:26 +04:00
parent a929cb3188
commit 935d64d9a9
1 changed files with 3 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/groovy/src/org/jetbrains/plugins/groovy/codeInspection/utils/EquivalenceChecker.java
+3 -28
View File
@@ -703,9 +703,9 @@ public class EquivalenceChecker {
private static boolean rangeExpressionsAreEquivalent(@NotNull GrRangeExpression rangeExp1,
@NotNull GrRangeExpression rangeExp2) {
return expressionsAreEquivalent(getLowerBoundForRange(rangeExp1), getLowerBoundForRange(rangeExp1)) &&
expressionsAreEquivalent(getUpperBoundForRange(rangeExp1), getUpperBoundForRange(rangeExp1)) &&
isInclusive(rangeExp1) == isInclusive(rangeExp2);
return expressionsAreEquivalent(rangeExp1.getLeftOperand(), rangeExp2.getLeftOperand()) &&
expressionsAreEquivalent(rangeExp1.getRightOperand(), rangeExp2.getRightOperand()) &&
isInclusive(rangeExp1) == isInclusive(rangeExp2);
}
private static boolean isInclusive(GrRangeExpression range) {
@@ -717,31 +717,6 @@ public class EquivalenceChecker {
return false;
}
@Nullable
private static GrExpression getLowerBoundForRange(GrRangeExpression range) {
for (PsiElement child : range.getChildren()) {
if (child instanceof GrExpression) {
return (GrExpression) child;
}
}
return null;
}
@Nullable
private static GrExpression getUpperBoundForRange(GrRangeExpression range) {
boolean firstChildSeen = false;
for (PsiElement child : range.getChildren()) {
if (child instanceof GrExpression) {
if (firstChildSeen) {
return (GrExpression) child;
} else {
firstChildSeen = true;
}
}
}
return null;
}
private static boolean assignmentExpressionsAreEquivalent(@NotNull GrAssignmentExpression assignExp1,
@NotNull GrAssignmentExpression assignExp2) {
final IElementType sign1 = assignExp1.getOperationToken();