diff --git a/plugins/InspectionGadgets/src/com/siyeh/ig/redundancy/RedundantStringOperationInspection.java b/plugins/InspectionGadgets/src/com/siyeh/ig/redundancy/RedundantStringOperationInspection.java index 1ab397cf7553..e964608e660c 100644 --- a/plugins/InspectionGadgets/src/com/siyeh/ig/redundancy/RedundantStringOperationInspection.java +++ b/plugins/InspectionGadgets/src/com/siyeh/ig/redundancy/RedundantStringOperationInspection.java @@ -315,7 +315,7 @@ public class RedundantStringOperationInspection extends AbstractBaseJavaLocalIns // s.lastIndexOf(..., s.length()) or s.lastIndexOf(..., s.length() - 1) if (stripped instanceof PsiBinaryExpression) { PsiBinaryExpression binOp = (PsiBinaryExpression)stripped; - if (binOp.getOperationTokenType().equals(JavaTokenType.MINUS) && + if (binOp.getOperationTokenType() == JavaTokenType.MINUS && ExpressionUtils.isLiteral(PsiUtil.skipParenthesizedExprDown(binOp.getROperand()), 1)) { stripped = binOp.getLOperand(); } @@ -672,7 +672,7 @@ public class RedundantStringOperationInspection extends AbstractBaseJavaLocalIns private static void extractSideEffects(PsiExpression result, PsiStatement statement) { List sideEffects = SideEffectChecker.extractSideEffectExpressions(result); - if (Collections.singletonList(result).equals(sideEffects)) return; + if (Objects.equals(Collections.singletonList(result), sideEffects)) return; PsiStatement[] statements = StatementExtractor.generateStatements(sideEffects, result); if (statements.length > 0) {