Remove ComparisonUtils duplication
authorBas Leijdekkers <basleijdekkers@gmail.com>
Wed, 5 Feb 2014 11:15:46 +0000 (12:15 +0100)
committerBas Leijdekkers <basleijdekkers@gmail.com>
Wed, 5 Feb 2014 11:17:45 +0000 (12:17 +0100)
plugins/IntentionPowerPak/src/com/siyeh/ipp/base/Intention.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/bool/ComparisonPredicate.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/bool/DemorgansIntention.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/bool/FlipComparisonIntention.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/bool/NegateComparisonIntention.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/expression/ExpressionPredicate.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/forloop/ReverseForLoopDirectionIntention.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/BoolUtils.java
plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ComparisonUtils.java [deleted file]
plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/VariableAccessUtils.java

index d26fc85b0c2b11a32b8e46a0608cd463a1d741c5..9e1ef7f75f7bbd82a0a5fe149b092ec6639a0209 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2003-2013 Dave Griffith, Bas Leijdekkers
+ * Copyright 2003-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,13 +22,10 @@ import com.intellij.openapi.editor.Editor;
 import com.intellij.openapi.project.Project;
 import com.intellij.psi.*;
 import com.intellij.psi.codeStyle.CodeStyleManager;
-import com.intellij.psi.codeStyle.JavaCodeStyleManager;
 import com.siyeh.IntentionPowerPackBundle;
-import com.siyeh.ig.InspectionGadgetsFix;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ig.psiutils.ParenthesesUtils;
 import com.siyeh.ipp.psiutils.BoolUtils;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
-import org.jetbrains.annotations.NonNls;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
index f8e39130f7c362d35913174d134f5e6d3d41f4ac..1712ea4572ec982fac578756103b43ea2bdc2455 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2003-2005 Dave Griffith
+ * Copyright 2003-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,8 +18,8 @@ package com.siyeh.ipp.bool;
 import com.intellij.psi.PsiBinaryExpression;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiExpression;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ipp.base.PsiElementPredicate;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
 import com.siyeh.ipp.psiutils.ErrorUtil;
 
 class ComparisonPredicate implements PsiElementPredicate {
index 6d7487523ac5057a6fbdf4ba24d5f263c6d8fe9c..eb27f3e51bf3c81f1a293f68ebee26c4a2b0c466 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2003-2013 Dave Griffith, Bas Leijdekkers
+ * Copyright 2003-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -19,11 +19,11 @@ import com.intellij.psi.*;
 import com.intellij.psi.tree.IElementType;
 import com.intellij.util.IncorrectOperationException;
 import com.siyeh.IntentionPowerPackBundle;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ig.psiutils.ParenthesesUtils;
 import com.siyeh.ipp.base.MutablyNamedIntention;
 import com.siyeh.ipp.base.PsiElementPredicate;
 import com.siyeh.ipp.psiutils.BoolUtils;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
 import org.jetbrains.annotations.NotNull;
 
 public class DemorgansIntention extends MutablyNamedIntention {
index 8f0d7936878feb04b1e6c759b480f8e56b1a94ed..97ea780f93f978e1035f5130eaa352ce8d6a9cd1 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2003-2009 Dave Griffith, Bas Leijdekkers
+ * Copyright 2003-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -23,9 +23,9 @@ import com.intellij.psi.tree.IElementType;
 import com.intellij.util.IncorrectOperationException;
 import com.siyeh.IntentionPowerPackBundle;
 import com.siyeh.ig.PsiReplacementUtil;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ipp.base.MutablyNamedIntention;
 import com.siyeh.ipp.base.PsiElementPredicate;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
 import org.jetbrains.annotations.NotNull;
 
 public class FlipComparisonIntention extends MutablyNamedIntention {
@@ -38,7 +38,7 @@ public class FlipComparisonIntention extends MutablyNamedIntention {
       final PsiJavaToken sign = expression.getOperationSign();
       operatorText = sign.getText();
       flippedOperatorText =
-        ComparisonUtils.getFlippedComparison(sign);
+        ComparisonUtils.getFlippedComparison(sign.getTokenType());
     }
     if (operatorText.equals(flippedOperatorText)) {
       return IntentionPowerPackBundle.message("flip.smth.intention.name",
index 2a5c9c413908291d467841f89e97350a55f07056..7844157e2186df0fbe7d9e285740b5cba21d0684 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2003-2006 Dave Griffith, Bas Leijdekkers
+ * Copyright 2003-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,10 +20,10 @@ import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiExpression;
 import com.intellij.psi.PsiJavaToken;
 import com.intellij.util.IncorrectOperationException;
+import com.siyeh.IntentionPowerPackBundle;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ipp.base.MutablyNamedIntention;
 import com.siyeh.ipp.base.PsiElementPredicate;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
-import com.siyeh.IntentionPowerPackBundle;
 import org.jetbrains.annotations.NotNull;
 
 public class NegateComparisonIntention extends MutablyNamedIntention {
index 7055f691077e85439b2f5821975155a9b879754f..b22f285b539c252fa6939ae50a7aa3fe070c9b88 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2007-2013 Dave Griffith, Bas Leijdekkers
+ * Copyright 2007-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  */
 package com.siyeh.ipp.expression;
 
-import com.intellij.psi.*;
+import com.intellij.psi.PsiElement;
+import com.intellij.psi.PsiExpression;
+import com.intellij.psi.PsiJavaToken;
+import com.intellij.psi.PsiPolyadicExpression;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ipp.base.PsiElementPredicate;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
 
 class ExpressionPredicate implements PsiElementPredicate {
 
index 92a3df479ef4e917fb7c679e6a62a9caf2cf7471..35404ae2f77dde264f807475010d2e32d4e6e527 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2009-2013 Bas Leijdekkers
+ * Copyright 2009-2014 Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -19,11 +19,11 @@ import com.intellij.openapi.project.Project;
 import com.intellij.psi.*;
 import com.intellij.psi.tree.IElementType;
 import com.intellij.util.IncorrectOperationException;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ig.psiutils.ExpressionUtils;
 import com.siyeh.ig.psiutils.ParenthesesUtils;
 import com.siyeh.ipp.base.Intention;
 import com.siyeh.ipp.base.PsiElementPredicate;
-import com.siyeh.ipp.psiutils.ComparisonUtils;
 import com.siyeh.ipp.psiutils.VariableAccessUtils;
 import org.jetbrains.annotations.NotNull;
 
index b5e01947a58664461e2583aaf605162cbcc485f5..295ca9a8df635d68f3d25d8967eba92e4c4cbcda 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2003-2013 Dave Griffith, Bas Leijdekkers
+ * Copyright 2003-2014 Dave Griffith, Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -17,6 +17,7 @@ package com.siyeh.ipp.psiutils;
 
 import com.intellij.psi.*;
 import com.intellij.psi.tree.IElementType;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ig.psiutils.ParenthesesUtils;
 import org.jetbrains.annotations.NonNls;
 import org.jetbrains.annotations.Nullable;
diff --git a/plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ComparisonUtils.java b/plugins/IntentionPowerPak/src/com/siyeh/ipp/psiutils/ComparisonUtils.java
deleted file mode 100644 (file)
index f2ffeda..0000000
+++ /dev/null
@@ -1,92 +0,0 @@
-/*
- * Copyright 2003-2009 Dave Griffith, Bas Leijdekkers
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package com.siyeh.ipp.psiutils;
-
-import com.intellij.psi.JavaTokenType;
-import com.intellij.psi.PsiBinaryExpression;
-import com.intellij.psi.PsiExpression;
-import com.intellij.psi.PsiJavaToken;
-import com.intellij.psi.tree.IElementType;
-import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
-
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.Map;
-import java.util.Set;
-
-public class ComparisonUtils {
-
-  private static final Set<IElementType> comparisons =
-    new HashSet<IElementType>(6);
-  private static final Map<IElementType, String> flippedComparisons =
-    new HashMap<IElementType, String>(6);
-  private static final Map<IElementType, String> negatedComparisons =
-    new HashMap<IElementType, String>(6);
-
-  private ComparisonUtils() {
-  }
-
-  static {
-    comparisons.add(JavaTokenType.EQEQ);
-    comparisons.add(JavaTokenType.NE);
-    comparisons.add(JavaTokenType.GT);
-    comparisons.add(JavaTokenType.LT);
-    comparisons.add(JavaTokenType.GE);
-    comparisons.add(JavaTokenType.LE);
-
-    flippedComparisons.put(JavaTokenType.EQEQ, "==");
-    flippedComparisons.put(JavaTokenType.NE, "!=");
-    flippedComparisons.put(JavaTokenType.GT, "<");
-    flippedComparisons.put(JavaTokenType.LT, ">");
-    flippedComparisons.put(JavaTokenType.GE, "<=");
-    flippedComparisons.put(JavaTokenType.LE, ">=");
-
-    negatedComparisons.put(JavaTokenType.EQEQ, "!=");
-    negatedComparisons.put(JavaTokenType.NE, "==");
-    negatedComparisons.put(JavaTokenType.GT, "<=");
-    negatedComparisons.put(JavaTokenType.LT, ">=");
-    negatedComparisons.put(JavaTokenType.GE, "<");
-    negatedComparisons.put(JavaTokenType.LE, ">");
-  }
-
-  public static boolean isComparison(@Nullable PsiExpression expression) {
-    if (!(expression instanceof PsiBinaryExpression)) {
-      return false;
-    }
-    final PsiBinaryExpression binaryExpression =
-      (PsiBinaryExpression)expression;
-    final IElementType tokenType = binaryExpression.getOperationTokenType();
-    return isComparison(tokenType);
-  }
-
-  public static boolean isComparison(@NotNull IElementType tokenType) {
-    return comparisons.contains(tokenType);
-  }
-
-  public static String getFlippedComparison(@NotNull PsiJavaToken sign) {
-    final IElementType text = sign.getTokenType();
-    return getFlippedComparison(text);
-  }
-
-  public static String getFlippedComparison(IElementType text) {
-    return flippedComparisons.get(text);
-  }
-
-  public static String getNegatedComparison(IElementType tokenType) {
-    return negatedComparisons.get(tokenType);
-  }
-}
\ No newline at end of file
index 46e221414a9160a3a9ee3ee53c50e6db51211990..2ac8500bad4a194261d89a9e6cc5770912042734 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2009-2013 Bas Leijdekkers
+ * Copyright 2009-2014 Bas Leijdekkers
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -17,6 +17,7 @@ package com.siyeh.ipp.psiutils;
 
 import com.intellij.psi.*;
 import com.intellij.psi.tree.IElementType;
+import com.siyeh.ig.psiutils.ComparisonUtils;
 import com.siyeh.ig.psiutils.ParenthesesUtils;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
@@ -39,7 +40,7 @@ public class VariableAccessUtils {
     final PsiBinaryExpression binaryExpression =
       (PsiBinaryExpression)expression;
     final IElementType tokenType = binaryExpression.getOperationTokenType();
-    if (!ComparisonUtils.isComparison(tokenType)) {
+    if (!ComparisonUtils.isComparisonOperation(tokenType)) {
       return false;
     }
     final PsiExpression lhs = binaryExpression.getLOperand();