IDEA-205482 "Use removal by object" fix may change semantics for List<Integer>

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantCollectionOperation/afterRemoveByIndexIntegerList.java

@@ -0,0 +1,8 @@
+// "Use removal by object" "true"
+import java.util.List;
+
+class Test {
+  void test(List<Integer> list, int key) {
+      list.remove((Integer) key);
+  }
+}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantCollectionOperation/beforeRemoveByIndexIntegerList.java

@@ -0,0 +1,9 @@
+// "Use removal by object" "true"
+import java.util.List;
+
+class Test {
+  void test(List<Integer> list, int key) {
+    int idx = list.indexOf(key);
+    list.r<caret>emove(idx);
+  }
+}

plugins/InspectionGadgets/src/com/siyeh/ig/redundancy/RedundantCollectionOperationInspection.java

@@ -131,6 +131,9 @@ public class RedundantCollectionOperationInspection extends AbstractBaseJavaLoca
       if (indexOfArg == null) return;
       CommentTracker ct = new CommentTracker();
       String text = ct.text(indexOfArg);
+      if (PsiType.INT.equals(indexOfArg.getType())) {
+        text = "(" + CommonClassNames.JAVA_LANG_INTEGER + ")" + text;
+      }
       if (!PsiTreeUtil.isAncestor(call, removeArg, false)) {
         PsiDeclarationStatement declaration = PsiTreeUtil.getParentOfType(removeArg, PsiDeclarationStatement.class);
         if (declaration == null) return;