KT-595 Don't create open functions in open classes

src/org/jetbrains/jet/j2k/Converter.java

@@ -274,6 +274,11 @@ public class Converter {
       modifiers.add(Modifier.OVERRIDE);
     if (method.getParent() instanceof PsiClass && ((PsiClass) method.getParent()).isInterface())
       modifiers.remove(Modifier.ABSTRACT);
+    if (method.getParent() instanceof PsiClass) {
+      final PsiModifierList parentModifierList = ((PsiClass) method.getParent()).getModifierList();
+      if (parentModifierList != null && parentModifierList.hasExplicitModifier(Modifier.FINAL))
+          modifiers.add(Modifier.NOT_OPEN);
+    }
 
     if (method.isConstructor()) { // TODO: simplify
       boolean isPrimary = isConstructorPrimary(method);

src/org/jetbrains/jet/j2k/ast/Function.java

@@ -78,6 +78,9 @@ public class Function extends Member {
     if (!myModifiers.contains(Modifier.OVERRIDE) && !myModifiers.contains(Modifier.FINAL))
       modifierList.add(Modifier.OPEN);
 
+    if (myModifiers.contains(Modifier.NOT_OPEN))
+      modifierList.remove(Modifier.OPEN);
+
     modifierList.add(accessModifier());
 
     if (modifierList.size() > 0)

src/org/jetbrains/jet/j2k/ast/Modifier.java

@@ -12,5 +12,6 @@ public abstract class Modifier {
   public static final String ABSTRACT = "abstract";
   public static final String FINAL = "final";
   public static final String OPEN = "open";
+  public static final String NOT_OPEN = "not open";
   public static final String OVERRIDE = "override";
 }

test/org/jetbrains/jet/j2k/JavaToKotlinConverterTest.java

@@ -114,7 +114,7 @@ public class JavaToKotlinConverterTest extends LightDaemonAnalyzerTestCase {
   static String statementToKotlin(String text) throws Exception {
     String result = methodToKotlin("void main() {" + text + "}");
     int pos = result.lastIndexOf("}");
-    result = result.substring(0, pos).replaceFirst("open fun main\\(\\) : Unit \\{", "");
+    result = result.substring(0, pos).replaceFirst("fun main\\(\\) : Unit \\{", "");
     return prettify(result);
   }
 

testData/ast/arrayType/method/methodArrayArgs.kt

@@ -1,2 +1,2 @@
-open fun fromArrayToCollection(a : Array<Foo?>?) : Unit {
+fun fromArrayToCollection(a : Array<Foo?>?) : Unit {
 }

testData/ast/class/class/classWithEmptyMethods.kt

@@ -1,8 +1,8 @@
 class T() {
-open fun main() : Unit {
+fun main() : Unit {
 }
-open fun i() : Int {
+fun i() : Int {
 }
-open fun s() : String? {
+fun s() : String? {
 }
 }

testData/ast/class/class/oneStaticFieldOneNonStatic.kt

@@ -1,5 +1,5 @@
 class S() {
-open fun sB() : Boolean {
+fun sB() : Boolean {
 return true
 }
 class object {

testData/ast/class/class/oneStaticMethod.kt

@@ -1,6 +1,6 @@
 class S() {
 class object {
-open fun staticF() : Boolean {
+fun staticF() : Boolean {
 return true
 }
 }

testData/ast/class/class/oneStaticMethodOneNonStatic.kt

@@ -1,9 +1,9 @@
 class S() {
-open fun sB() : Boolean {
+fun sB() : Boolean {
 return true
 }
 class object {
-open fun sI() : Int {
+fun sI() : Int {
 return 1
 }
 }

testData/ast/class/class/twoStaticMethod.kt

@@ -1,9 +1,9 @@
 class S() {
 class object {
-open fun sB() : Boolean {
+fun sB() : Boolean {
 return true
 }
-open fun sI() : Int {
+fun sI() : Int {
 return 1
 }
 }

testData/ast/function/file/functionInFinalClass.jav

@@ -0,0 +1,5 @@
+package demo;
+
+final class Final {
+  void test() {}
+}

testData/ast/function/file/functionInFinalClass.kt

@@ -0,0 +1,5 @@
+namespace demo
+class Final() {
+fun test() : Unit {
+}
+}

testData/ast/function/method/abstractMethod.kt

@@ -1 +1 @@
-abstract open fun getNoofGears() : Int
+abstract fun getNoofGears() : Int

testData/ast/function/method/classGenericParam.kt

@@ -1,2 +1,2 @@
-open fun getT() : T? {
+fun getT() : T? {
 }

testData/ast/function/method/emptyVoidMethod.kt

@@ -1,2 +1,2 @@
-open fun main() : Unit {
+fun main() : Unit {
 }

testData/ast/function/method/internal.kt

@@ -1,2 +1,2 @@
-open fun test() : Unit {
+fun test() : Unit {
 }

testData/ast/function/method/main.kt

@@ -1,4 +1,4 @@
 class object {
-open public fun main(args : Array<String?>?) : Unit {
+public fun main(args : Array<String?>?) : Unit {
 }
 }

testData/ast/function/method/methodClassType.kt

@@ -1,2 +1,2 @@
-open fun main() : String? {
+fun main() : String? {
 }

testData/ast/function/method/methodPrimitiveType.kt

@@ -1,2 +1,2 @@
-open fun main() : Int {
+fun main() : Int {
 }

testData/ast/function/method/methodPrimitiveType2.kt

@@ -1,2 +1,2 @@
-open fun main() : Boolean {
+fun main() : Boolean {
 }

testData/ast/function/method/methodWithReturnStatement.kt

@@ -1,3 +1,3 @@
-open fun isTrue() : Boolean {
+fun isTrue() : Boolean {
 return true
 }

testData/ast/function/method/open.kt

@@ -1,3 +1,3 @@
-open fun getString() : String? {
+fun getString() : String? {
 return ""
 }

testData/ast/function/method/ownGenericParam.kt

@@ -1,2 +1,2 @@
-open fun putU<U>(u : U?) : Unit {
+fun putU<U>(u : U?) : Unit {
 }

testData/ast/function/method/ownSeveralGenericParams.kt

@@ -1,2 +1,2 @@
-open fun putUVW<U, V, W>(u : U?, v : V?, w : W?) : Unit {
+fun putUVW<U, V, W>(u : U?, v : V?, w : W?) : Unit {
 }

testData/ast/function/method/private.kt

@@ -1,2 +1,2 @@
-open private fun test() : Unit {
+private fun test() : Unit {
 }

testData/ast/function/method/protected.kt

@@ -1,2 +1,2 @@
-open protected fun test() : Unit {
+protected fun test() : Unit {
 }

testData/ast/function/method/public.kt

@@ -1,2 +1,2 @@
-open public fun test() : Unit {
+public fun test() : Unit {
 }

testData/ast/inProjectionType/method/methodParams.kt

@@ -1,2 +1,2 @@
-open fun popAll(dst : Collection<in E?>?) : Unit {
+fun popAll(dst : Collection<in E?>?) : Unit {
 }

testData/ast/outProjectionType/method/methodParams.kt

@@ -1,2 +1,2 @@
-open fun pushAll(src : Collection<out E?>?) : Unit {
+fun pushAll(src : Collection<out E?>?) : Unit {
 }

testData/ast/starProjectionType/method/methodParams.kt

@@ -1,2 +1,2 @@
-open fun wtf(w : Collection<*>?) : Unit {
+fun wtf(w : Collection<*>?) : Unit {
 }

testData/ast/typeParameters/method/methodDoubleParametrizationWithTwoBounds.kt

@@ -1,2 +1,2 @@
-open fun max<T : Any?, K : Node?>(coll : Collection<out T?>?) : T? where T : Comparable<in T?>?, K : Collection<in K?>? {
+fun max<T : Any?, K : Node?>(coll : Collection<out T?>?) : T? where T : Comparable<in T?>?, K : Collection<in K?>? {
 }

testData/ast/typeParameters/method/where.kt

@@ -1,2 +1,2 @@
-open fun max<T : Any?>(coll : Collection<out T?>?) : T? where T : Comparable<in T?>? {
+fun max<T : Any?>(coll : Collection<out T?>?) : T? where T : Comparable<in T?>? {
 }

testData/ast/varArg/method/ellipsisTypeSeveralParams.kt

@@ -1 +1 @@
-open fun format(pattern : String?, vararg arguments : Any?) : String?
+fun format(pattern : String?, vararg arguments : Any?) : String?

testData/ast/varArg/method/ellipsisTypeSingleParams.kt

@@ -1,2 +1,2 @@
-open fun pushAll(vararg objs : Any?) : Unit {
+fun pushAll(vararg objs : Any?) : Unit {
 }