cleanup
authorAlexey Kudravtsev <cdr@intellij.com>
Thu, 23 Jun 2011 11:16:40 +0000 (15:16 +0400)
committerAlexey Kudravtsev <cdr@intellij.com>
Thu, 23 Jun 2011 12:13:29 +0000 (16:13 +0400)
22 files changed:
java/debugger/impl/src/com/intellij/debugger/engine/evaluation/expression/EvaluatorBuilderImpl.java
java/java-impl/src/com/intellij/codeInsight/daemon/impl/analysis/HighlightControlFlowUtil.java
java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/CreateLocalVarFromInstanceofAction.java
java/java-impl/src/com/intellij/codeInsight/editorActions/DeclarationJoinLinesHandler.java
java/java-impl/src/com/intellij/codeInsight/intention/impl/ConcatenationToMessageFormatAction.java
java/java-impl/src/com/intellij/codeInsight/intention/impl/SplitIfAction.java
java/java-impl/src/com/intellij/codeInspection/dataFlow/ControlFlowAnalyzer.java
java/java-impl/src/com/intellij/codeInspection/dataFlow/DataFlowInspection.java
java/java-impl/src/com/intellij/codeInspection/defUse/DefUseInspection.java
java/java-impl/src/com/intellij/codeInspection/sillyAssignment/SillyAssignmentInspection.java
java/java-impl/src/com/intellij/psi/filters/getters/JavaMembersGetter.java
java/java-impl/src/com/intellij/psi/formatter/java/FormattingAstUtil.java
java/java-impl/src/com/intellij/psi/impl/source/tree/java/PsiLiteralExpressionImpl.java
java/java-impl/src/com/intellij/refactoring/invertBoolean/InvertBooleanProcessor.java
java/java-impl/src/com/intellij/refactoring/typeMigration/TypeMigrationStatementProcessor.java
java/openapi/src/com/intellij/psi/util/IsConstantExpressionVisitor.java
java/openapi/src/com/intellij/psi/util/PsiUtil.java
java/openapi/src/com/intellij/psi/util/RedundantCastUtil.java
plugins/IntelliLang/java-support/org/intellij/plugins/intelliLang/util/AnnotationUtilEx.java
plugins/IntelliLang/java-support/org/intellij/plugins/intelliLang/util/ContextComputationProcessor.java
plugins/java-i18n/src/com/intellij/codeInspection/i18n/I18nizeConcatenationQuickFix.java
samples/comparingReferences/source/com/intellij/codeInspection/ComparingReferencesInspection.java

index 108c3cf2807517edb35c5b73c8e672ba3ca0594b..7c944c54d9b19743afa642fc6600b6b0c9e88948 100644 (file)
@@ -130,7 +130,7 @@ public class EvaluatorBuilderImpl implements EvaluatorBuilder {
       rExpression.accept(this);
       Evaluator rEvaluator = myResult;
 
-      if(expression.getOperationSign().getTokenType() != JavaTokenType.EQ) {
+      if(expression.getOperationTokenType() != JavaTokenType.EQ) {
         throwEvaluateException(DebuggerBundle.message("evaluation.error.operation.not.supported", expression.getOperationSign().getText()));
       }
 
@@ -311,7 +311,7 @@ public class EvaluatorBuilderImpl implements EvaluatorBuilder {
       }
       rOperand.accept(this);
       Evaluator rResult = myResult;
-      IElementType opType = expression.getOperationSign().getTokenType();
+      IElementType opType = expression.getOperationTokenType();
       PsiType expressionExpectedType = expression.getType();
       if (expressionExpectedType == null) {
         throwEvaluateException(DebuggerBundle.message("evaluation.error.unknown.expression.type", expression.getText()));
index 1785837da3a29d099d0069ba3cd8c0ba17b7eb5d..94f379eb94f7b785b523c16fbec11a0b5320b1d8 100644 (file)
@@ -589,14 +589,14 @@ public class HighlightControlFlowUtil {
     }
     else if (expression instanceof PsiPostfixExpression) {
       final PsiExpression operand = ((PsiPostfixExpression)expression).getOperand();
-      final IElementType sign = ((PsiPostfixExpression)expression).getOperationSign().getTokenType();
+      final IElementType sign = ((PsiPostfixExpression)expression).getOperationTokenType();
       if (operand instanceof PsiReferenceExpression && (sign == JavaTokenType.PLUSPLUS || sign == JavaTokenType.MINUSMINUS)) {
         reference = (PsiReferenceExpression)operand;
       }
     }
     else if (expression instanceof PsiPrefixExpression) {
       final PsiExpression operand = ((PsiPrefixExpression)expression).getOperand();
-      final IElementType sign = ((PsiPrefixExpression)expression).getOperationSign().getTokenType();
+      final IElementType sign = ((PsiPrefixExpression)expression).getOperationTokenType();
       if (operand instanceof PsiReferenceExpression && (sign == JavaTokenType.PLUSPLUS || sign == JavaTokenType.MINUSMINUS)) {
         reference = (PsiReferenceExpression)operand;
       }
index c81b0044d9292443e5b0dfed56d9207c96a904db..3e0e9b7919269d41c29ccc3faf616911a495592e 100644 (file)
@@ -294,7 +294,7 @@ public class CreateLocalVarFromInstanceofAction extends BaseIntentionAction {
     while (element instanceof PsiParenthesizedExpression) {
       element = element.getParent();
     }
-    return element instanceof PsiPrefixExpression && ((PsiPrefixExpression)element).getOperationSign().getTokenType() == JavaTokenType.EXCL;
+    return element instanceof PsiPrefixExpression && ((PsiPrefixExpression)element).getOperationTokenType() == JavaTokenType.EXCL;
   }
 
   private static Template generateTemplate(Project project, PsiExpression initializer, PsiType type) {
index 3a84de52c010e2a061bd8796bf3ea3eeedf9f715..5cc8800dd41589f2a5e58a1aec00cbf25c765f0e 100644 (file)
@@ -64,7 +64,7 @@ public class DeclarationJoinLinesHandler implements JoinLinesHandlerDelegate {
 
     final PsiElementFactory factory = JavaPsiFacade.getInstance(psiManager.getProject()).getElementFactory();
     PsiExpression initializerExpression;
-    final IElementType originalOpSign = assignment.getOperationSign().getTokenType();
+    final IElementType originalOpSign = assignment.getOperationTokenType();
     if (originalOpSign == JavaTokenType.EQ) {
       initializerExpression = assignment.getRExpression();
     }
index b5ef45ce60e2d624bcee927086a006f42a1f9f58..eb0ea9d98b508d9bf02743afc2cec9ac79c8ee04 100644 (file)
@@ -89,7 +89,7 @@ public class ConcatenationToMessageFormatAction implements IntentionAction {
     if (expression instanceof PsiBinaryExpression) {
       final PsiType type = expression.getType();
       if (type != null && type.equalsToText("java.lang.String")
-          && ((PsiBinaryExpression)expression).getOperationSign().getTokenType() == JavaTokenType.PLUS) {
+          && ((PsiBinaryExpression)expression).getOperationTokenType() == JavaTokenType.PLUS) {
         buildMessageFormatString(((PsiBinaryExpression)expression).getLOperand(), formatString, args);
         final PsiExpression rhs = ((PsiBinaryExpression)expression).getROperand();
         if (rhs != null) {
index 2b2cd55374d8db864a9db3d3bc8eb2ca117fafef..15b9f6f66d48571c104f0cbf75eaee5b7e15ef74 100644 (file)
@@ -44,14 +44,14 @@ public class SplitIfAction extends PsiElementBaseIntentionAction {
     if (!(token.getParent() instanceof PsiBinaryExpression)) return false;
 
     PsiBinaryExpression expression = (PsiBinaryExpression)token.getParent();
-    boolean isAndExpression = expression.getOperationSign().getTokenType() == JavaTokenType.ANDAND;
-    boolean isOrExpression = expression.getOperationSign().getTokenType() == JavaTokenType.OROR;
+    boolean isAndExpression = expression.getOperationTokenType() == JavaTokenType.ANDAND;
+    boolean isOrExpression = expression.getOperationTokenType() == JavaTokenType.OROR;
     if (!isAndExpression && !isOrExpression) return false;
 
     while (expression.getParent() instanceof PsiBinaryExpression) {
       expression = (PsiBinaryExpression)expression.getParent();
-      if (isAndExpression && expression.getOperationSign().getTokenType() != JavaTokenType.ANDAND) return false;
-      if (isOrExpression && expression.getOperationSign().getTokenType() != JavaTokenType.OROR) return false;
+      if (isAndExpression && expression.getOperationTokenType() != JavaTokenType.ANDAND) return false;
+      if (isOrExpression && expression.getOperationTokenType() != JavaTokenType.OROR) return false;
     }
 
     if (!(expression.getParent() instanceof PsiIfStatement)) return false;
index b1afd83a6068a443ee70c1aeead086685b1d6efd..128980674a03bf01782407e6d836eb6ce968948c 100644 (file)
@@ -145,7 +145,7 @@ class ControlFlowAnalyzer extends JavaElementVisitor {
 
       lExpr.accept(this);
 
-      IElementType op = expression.getOperationSign().getTokenType();
+      IElementType op = expression.getOperationTokenType();
       PsiType type = expression.getType();
       boolean isBoolean = PsiType.BOOLEAN.equals(type);
       if (op == JavaTokenType.EQ) {
@@ -882,7 +882,7 @@ class ControlFlowAnalyzer extends JavaElementVisitor {
         addInstruction(new PushInstruction(dfaValue, expression));
       }
       else {
-        IElementType op = expression.getOperationSign().getTokenType();
+        IElementType op = expression.getOperationTokenType();
         PsiExpression lExpr = expression.getLOperand();
         PsiExpression rExpr = expression.getROperand();
 
index c40caa7a03a8a3af8181c988f9fe8e1030b6912f..351907fc9a39bdc3fd9e5a7b4474d7dfd2f3e494 100644 (file)
@@ -289,7 +289,7 @@ public class DataFlowInspection extends BaseLocalInspectionTool {
     if (!field.hasModifierProperty(PsiModifier.FINAL)) return false;
 
     PsiElement parent = element.getParent();
-    if (parent instanceof PsiPrefixExpression && ((PsiPrefixExpression)parent).getOperationSign().getTokenType() == JavaTokenType.EXCL) {
+    if (parent instanceof PsiPrefixExpression && ((PsiPrefixExpression)parent).getOperationTokenType() == JavaTokenType.EXCL) {
       element = parent;
       parent = parent.getParent();
     }
index 530363520bcb6c094d3d002cef14b046f38a49cf..eeca032ba4ddd11f1e3c44ebf8cededd88b502a9 100644 (file)
@@ -117,7 +117,7 @@ public class DefUseInspection extends BaseLocalInspectionTool {
           }
         }
         else if (context instanceof PsiAssignmentExpression &&
-                 ((PsiAssignmentExpression)context).getOperationSign().getTokenType() == JavaTokenType.EQ) {
+                 ((PsiAssignmentExpression)context).getOperationTokenType() == JavaTokenType.EQ) {
           final PsiAssignmentExpression assignment = (PsiAssignmentExpression)context;
           holder.registerProblem(assignment.getRExpression(),
                                  InspectionsBundle.message("inspection.unused.assignment.problem.descriptor3",
index 68fbe8205ceb7adc1663b374b39e40c1d56403af..c823a3cd344043999340084dfd3388bb02a5dd89 100644 (file)
@@ -80,7 +80,7 @@ public class SillyAssignmentInspection extends BaseJavaLocalInspectionTool {
   }
 
   private static void checkSillyAssignment(PsiAssignmentExpression assignment, ProblemsHolder holder) {
-    if (assignment.getOperationSign().getTokenType() != JavaTokenType.EQ) return;
+    if (assignment.getOperationTokenType() != JavaTokenType.EQ) return;
     PsiExpression lExpression = assignment.getLExpression();
     PsiExpression rExpression = assignment.getRExpression();
     if (rExpression == null) return;
index 9963cc3ac9686ed15d484f999c23b003e914701d..c6b932ae50160f1f1d39963b987994f62fac50f7 100644 (file)
@@ -57,7 +57,7 @@ public class JavaMembersGetter extends MembersGetter {
     parent = parent.getParent();
     while (parent instanceof PsiBinaryExpression) {
       final PsiBinaryExpression binaryExpression = (PsiBinaryExpression)parent;
-      final IElementType op = binaryExpression.getOperationSign().getTokenType();
+      final IElementType op = binaryExpression.getOperationTokenType();
       if (JavaTokenType.EQEQ == op || JavaTokenType.NE == op) {
         if (prev == binaryExpression.getROperand()) {
           processMembers(position, results, getCalledClass(binaryExpression.getLOperand()), false);
index 3955e78abd317c5645a0d91da98298322902a082..2d7c7027160294e97460c5c67dc795af94a4e14d 100644 (file)
@@ -153,6 +153,6 @@ public class FormattingAstUtil {
     }
     PsiBinaryExpression expression1 = (PsiBinaryExpression)SourceTreeToPsiMap.treeElementToPsi(node1);
     PsiBinaryExpression expression2 = (PsiBinaryExpression)SourceTreeToPsiMap.treeElementToPsi(node2);
-    return expression1.getOperationSign().getTokenType() == expression2.getOperationSign().getTokenType();
+    return expression1.getOperationTokenType() == expression2.getOperationTokenType();
   }
 }
index 52959c6b224850c6f969545d164c76b24c83711d..f5b13ded13ae64022241ecb103ec076cf705126c 100644 (file)
@@ -262,7 +262,7 @@ public class PsiLiteralExpressionImpl
       //literal 2147483648 may appear only as the operand of the unary negation operator -.
       if (!(text.equals(_2_IN_31)
             && getParent() instanceof PsiPrefixExpression
-            && ((PsiPrefixExpression)getParent()).getOperationSign().getTokenType() == JavaTokenType.MINUS)) {
+            && ((PsiPrefixExpression)getParent()).getOperationTokenType() == JavaTokenType.MINUS)) {
         if (text.equals(HEX_PREFIX)) {
           return JavaErrorMessages.message("hexadecimal.numbers.must.contain.at.least.one.hexadecimal.digit");
         }
@@ -278,7 +278,7 @@ public class PsiLiteralExpressionImpl
       //literal 9223372036854775808L may appear only as the operand of the unary negation operator -.
       if (!(text.equals(_2_IN_63_L)
             && getParent() instanceof PsiPrefixExpression
-            && ((PsiPrefixExpression)getParent()).getOperationSign().getTokenType() == JavaTokenType.MINUS)) {
+            && ((PsiPrefixExpression)getParent()).getOperationTokenType() == JavaTokenType.MINUS)) {
         if (text.equals(LONG_HEX_EMPTY)) {
           return JavaErrorMessages.message("hexadecimal.numbers.must.contain.at.least.one.hexadecimal.digit");
         }
index 7dac17b42d051a51c46c478ae248fb0613badb3a..fedf8fe1da4db6cf4f2c551fdbbc36432c7399f9 100644 (file)
@@ -238,7 +238,7 @@ public class InvertBooleanProcessor extends BaseRefactoringProcessor {
         if (expression.getParent() instanceof PsiMethodCallExpression) expression = (PsiExpression)expression.getParent();
         try {
           while (expression.getParent() instanceof PsiPrefixExpression &&
-                 ((PsiPrefixExpression)expression.getParent()).getOperationSign().getTokenType() == JavaTokenType.EXCL) {
+                 ((PsiPrefixExpression)expression.getParent()).getOperationTokenType() == JavaTokenType.EXCL) {
             expression = (PsiExpression)expression.getParent();
           }
           if (!(expression.getParent() instanceof PsiExpressionStatement)) {
index 7214fa87f1f04ef8583fafb6438a2af3015ea6a0..d20f81fad96ca113dadd9c530ddc2409d7b4b1ba 100644 (file)
@@ -351,7 +351,7 @@ class TypeMigrationStatementProcessor extends JavaRecursiveElementVisitor {
     final PsiExpression rOperand = expression.getROperand();
     if (rOperand == null) return;
     final TypeView right = new TypeView(rOperand);
-    if (!TypeConversionUtil.isBinaryOperatorApplicable(expression.getOperationSign().getTokenType(), left.getType(), right.getType(), false)) {
+    if (!TypeConversionUtil.isBinaryOperatorApplicable(expression.getOperationTokenType(), left.getType(), right.getType(), false)) {
       if (left.isChanged()) {
         findConversionOrFail(expression, lOperand, left.getTypePair());
       }
index 0d8ba082d64cf5cbc81a1e91253846102429a942..79723f2a7462d56a4083788bd6f531dac8d160c7 100644 (file)
@@ -75,7 +75,7 @@ public class IsConstantExpressionVisitor extends JavaElementVisitor {
     }
     operand.accept(this);
     if (!myIsConstant) return;
-    IElementType opType = expression.getOperationSign().getTokenType();
+    IElementType opType = expression.getOperationTokenType();
     if (opType == JavaTokenType.PLUS || opType == JavaTokenType.MINUS || opType == JavaTokenType.TILDE || opType == JavaTokenType.EXCL) {
       return;
     }
index 180b3ea2e275ea486608ac8b8be90523558e5631..cc5d074638b4fe9822bad23cd57455b208d3449d 100644 (file)
@@ -81,7 +81,7 @@ public final class PsiUtil extends PsiUtilBase {
     PsiElement parent = PsiTreeUtil.skipParentsOfType(expr, PsiParenthesizedExpression.class);
     return !(parent instanceof PsiAssignmentExpression) ||
            !PsiTreeUtil.isAncestor(((PsiAssignmentExpression)parent).getLExpression(), expr, false) ||
-           ((PsiAssignmentExpression)parent).getOperationSign().getTokenType() != JavaTokenType.EQ;
+           ((PsiAssignmentExpression)parent).getOperationTokenType() != JavaTokenType.EQ;
   }
 
   public static boolean isAccessible(PsiMember member, @NotNull PsiElement place, @Nullable PsiClass accessObjectClass) {
@@ -276,12 +276,12 @@ public final class PsiUtil extends PsiUtilBase {
 
   public static boolean isIncrementDecrementOperation(PsiElement element) {
     if (element instanceof PsiPostfixExpression) {
-      final IElementType sign = ((PsiPostfixExpression) element).getOperationSign().getTokenType();
+      final IElementType sign = ((PsiPostfixExpression)element).getOperationTokenType();
       if (sign == JavaTokenType.PLUSPLUS || sign == JavaTokenType.MINUSMINUS)
         return true;
     }
     else if (element instanceof PsiPrefixExpression) {
-      final IElementType sign = ((PsiPrefixExpression) element).getOperationSign().getTokenType();
+      final IElementType sign = ((PsiPrefixExpression)element).getOperationTokenType();
       if (sign == JavaTokenType.PLUSPLUS || sign == JavaTokenType.MINUSMINUS)
         return true;
     }
index 310342495d9549fdc34dd1ad2e22e1ce7c759d52..8cb12c23b818ef4261104e98cc5aeb0deaebdf59 100644 (file)
@@ -141,7 +141,7 @@ public class RedundantCastUtil {
       PsiExpression lExpr = deparenthesizeExpression(expression.getROperand());
 
       if (rExpr != null && lExpr != null) {
-        final IElementType binaryToken = expression.getOperationSign().getTokenType();
+        final IElementType binaryToken = expression.getOperationTokenType();
         processBinaryExpressionOperand(lExpr, rExpr, binaryToken);
         processBinaryExpressionOperand(rExpr, lExpr, binaryToken);
       }
index 2ca796d00f5419127f5e26b40420c20506a0e719..fd30cc70dd8d641b2ae48ef9726fea510d222cbb 100644 (file)
@@ -69,7 +69,7 @@ public class AnnotationUtilEx {
       element = ContextComputationProcessor.getTopLevelInjectionTarget(element);
       final PsiElement parent = element.getParent();
 
-      if (element instanceof PsiAssignmentExpression && ((PsiAssignmentExpression)element).getOperationSign().getTokenType() == JavaTokenType.PLUSEQ) {
+      if (element instanceof PsiAssignmentExpression && ((PsiAssignmentExpression)element).getOperationTokenType() == JavaTokenType.PLUSEQ) {
         element = ((PsiAssignmentExpression)element).getLExpression();
         continue;
       }
@@ -145,7 +145,7 @@ public class AnnotationUtilEx {
 
     if (parent instanceof PsiAssignmentExpression) {
       final PsiAssignmentExpression p = (PsiAssignmentExpression)parent;
-      if (p.getRExpression() == element || p.getOperationSign().getTokenType() == JavaTokenType.PLUSEQ) {
+      if (p.getRExpression() == element || p.getOperationTokenType() == JavaTokenType.PLUSEQ) {
         final PsiExpression left = p.getLExpression();
         if (left instanceof PsiReferenceExpression) {
           if (!visitor.visitReference((PsiReferenceExpression)left)) return false;
index b44d61877ca70ab4969fe60312333d3ad4277d7a..2bc55e1e6fa705588138be222b8011015fc146e4 100644 (file)
@@ -77,13 +77,13 @@ public class ContextComputationProcessor {
       collectOperands(((PsiConditionalExpression)expression).getElseExpression(), result, unparsable);
     }
     else if (expression instanceof PsiBinaryExpression &&
-             ((PsiBinaryExpression)expression).getOperationSign().getTokenType() == JavaTokenType.PLUS) {
+             ((PsiBinaryExpression)expression).getOperationTokenType() == JavaTokenType.PLUS) {
       final PsiBinaryExpression binaryExpression = (PsiBinaryExpression)expression;
       collectOperands(binaryExpression.getLOperand(), result, unparsable);
       collectOperands(binaryExpression.getROperand(), result, unparsable);
     }
     else if (expression instanceof PsiAssignmentExpression &&
-             ((PsiAssignmentExpression)expression).getOperationSign().getTokenType() == JavaTokenType.PLUSEQ) {
+             ((PsiAssignmentExpression)expression).getOperationTokenType() == JavaTokenType.PLUSEQ) {
       unparsable.set(Boolean.TRUE);
       final PsiAssignmentExpression assignmentExpression = (PsiAssignmentExpression)expression;
       collectOperands(assignmentExpression.getLExpression(), result, unparsable);
index da7ee1fcea57162e1443d6617f59c2abdd6bb260..7528eca7cba4620fec17866dd513585570cc0bc9 100644 (file)
@@ -125,7 +125,7 @@ public class I18nizeConcatenationQuickFix extends I18nizeQuickFix{
     while (true) {
       PsiExpression lOperand = element.getLOperand();
       PsiExpression rOperand = element.getROperand();
-      if (element.getOperationSign().getTokenType() != JavaTokenType.PLUS) return concatenation;
+      if (element.getOperationTokenType() != JavaTokenType.PLUS) return concatenation;
       stringLiteralOccured |= lOperand instanceof PsiLiteralExpression && ((PsiLiteralExpression)lOperand).getValue() instanceof String ||
                               rOperand instanceof PsiLiteralExpression && ((PsiLiteralExpression)rOperand).getValue() instanceof String;
 
index 6410db394eed00df86762123adc2e8493ee0c36a..941fc702cae4e72c36b451c9542d9dcadaf12d58 100644 (file)
@@ -69,7 +69,7 @@ public class ComparingReferencesInspection extends BaseJavaLocalInspectionTool {
 
       @Override public void visitBinaryExpression(PsiBinaryExpression expression) {
           super.visitBinaryExpression(expression);
-          IElementType opSign = expression.getOperationSign().getTokenType();
+        IElementType opSign = expression.getOperationTokenType();
           if (opSign == JavaTokenType.EQEQ || opSign == JavaTokenType.NE) {
               PsiExpression lOperand = expression.getLOperand();
               PsiExpression rOperand = expression.getROperand();
@@ -102,7 +102,7 @@ public class ComparingReferencesInspection extends BaseJavaLocalInspectionTool {
     public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descriptor) {
       try {
         PsiBinaryExpression binaryExpression = (PsiBinaryExpression)descriptor.getPsiElement();
-        IElementType opSign = binaryExpression.getOperationSign().getTokenType();
+        IElementType opSign = binaryExpression.getOperationTokenType();
         PsiExpression lExpr = binaryExpression.getLOperand();
         PsiExpression rExpr = binaryExpression.getROperand();
         if (rExpr == null)