Introduce "Redundant 'inner' modifier" inspection

#KT-3262 Fixed

idea/testData/inspectionsLocal/redundantInnerClassModifier/.inspection

@@ -0,0 +1 @@
+org.jetbrains.kotlin.idea.inspections.RedundantInnerClassModifierInspection

idea/testData/inspectionsLocal/redundantInnerClassModifier/extendClass.kt

@@ -0,0 +1,7 @@
+class A {
+    fun a() {}
+
+    open class B
+
+    <caret>inner class C : B()
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/extendClass.kt.after

@@ -0,0 +1,7 @@
+class A {
+    fun a() {}
+
+    open class B
+
+    class C : B()
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/extendClass2.kt

@@ -0,0 +1,7 @@
+class A {
+    fun a() {}
+
+    open class B(i: Int)
+
+    <caret>inner class C(i: Int) : B(i)
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/extendClass2.kt.after

@@ -0,0 +1,7 @@
+class A {
+    fun a() {}
+
+    open class B(i: Int)
+
+    class C(i: Int) : B(i)
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/extendInnerClass.kt

@@ -0,0 +1,8 @@
+// PROBLEM: none
+class A {
+    fun a() {}
+
+    open inner class B
+
+    <caret>inner class C : B()
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/extendInnerClass2.kt

@@ -0,0 +1,8 @@
+// PROBLEM: none
+class A {
+    fun a() {}
+
+    open inner class B(i: Int)
+
+    <caret>inner class C(i: Int) : B(i)
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasOuterClassMemberReference.kt

@@ -0,0 +1,10 @@
+// PROBLEM: none
+class A {
+    fun a() {}
+
+    <caret>inner class B {
+        fun b() {
+            a()
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasOuterClassMemberReference2.kt

@@ -0,0 +1,10 @@
+// PROBLEM: none
+class A {
+    val a = 1
+
+    <caret>inner class B {
+        fun b() {
+            a
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasOuterClassMemberReference3.kt

@@ -0,0 +1,13 @@
+// PROBLEM: none
+class A {
+    fun a() {}
+
+    inner class B {
+
+        <caret>inner class C {
+            fun c() {
+                a()
+            }
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasOuterClassMemberReference4.kt

@@ -0,0 +1,10 @@
+// PROBLEM: none
+class A {
+    fun a() {}
+
+    <caret>inner class B {
+        fun b() {
+            this@A.a()
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasOuterClassMemberReference5.kt

@@ -0,0 +1,10 @@
+// PROBLEM: none
+class A {
+    fun a() {}
+
+    <caret>inner class B {
+        fun b() {
+            this@A
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasOuterClassTypeReference.kt

@@ -0,0 +1,7 @@
+// PROBLEM: none
+class A<T> {
+    <caret>inner class B {
+        fun b(t: T) {
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasSuperClassMemberReference.kt

@@ -0,0 +1,14 @@
+// PROBLEM: none
+open class A {
+    fun a() {}
+}
+
+open class B : A()
+
+class C : B() {
+    <caret>inner class D {
+        fun d() {
+            a()
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasSuperJavaClassMemberReference.1.java

@@ -0,0 +1,5 @@
+public class Java {
+    public String getText() {
+        return "";
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasSuperJavaClassMemberReference.kt

@@ -0,0 +1,8 @@
+// PROBLEM: none
+class A : Java() {
+    <caret>inner class B {
+        fun b() {
+            getText()
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasSuperJavaClassMemberReference2.1.java

@@ -0,0 +1,5 @@
+public class Java {
+    public String getText() {
+        return "";
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/hasSuperJavaClassMemberReference2.kt

@@ -0,0 +1,8 @@
+// PROBLEM: none
+class A : Java() {
+    <caret>inner class B {
+        fun b() {
+            text
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/inInnerClass.kt

@@ -0,0 +1,6 @@
+// PROBLEM: none
+class A {
+    inner class B {
+        <caret>inner class C
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/inLocalClass.kt

@@ -0,0 +1,6 @@
+// PROBLEM: none
+fun test() {
+    class A {
+        <caret>inner class B
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassMemberReference.kt

@@ -0,0 +1,3 @@
+class A {
+    <caret>inner class B
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassMemberReference.kt.after

@@ -0,0 +1,3 @@
+class A {
+    class B
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassMemberReference2.kt

@@ -0,0 +1,9 @@
+fun foo() {}
+
+class A {
+    <caret>inner class B {
+        fun b() {
+            foo()
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassMemberReference2.kt.after

@@ -0,0 +1,9 @@
+fun foo() {}
+
+class A {
+    class B {
+        fun b() {
+            foo()
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassMemberReference3.kt

@@ -0,0 +1,12 @@
+class A {
+    <caret>inner class B {
+
+        fun b() {}
+
+        inner class C {
+            fun c() {
+                b()
+            }
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassMemberReference3.kt.after

@@ -0,0 +1,12 @@
+class A {
+    class B {
+
+        fun b() {}
+
+        inner class C {
+            fun c() {
+                b()
+            }
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassTypeReference.kt

@@ -0,0 +1,6 @@
+class A {
+    <caret>inner class B<T> {
+        fun b(t: T) {
+        }
+    }
+}

idea/testData/inspectionsLocal/redundantInnerClassModifier/noOuterClassTypeReference.kt.after

@@ -0,0 +1,6 @@
+class A {
+    class B<T> {
+        fun b(t: T) {
+        }
+    }
+}