open modifier for functions added

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

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

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

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

testData/ast/arrayType/method/methodArrayArgs.kt

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

testData/ast/callChainExpression/file/libraryFieldCall.kt

@@ -5,7 +5,7 @@ val ourOut : PrintStream?
 }
 }
 open class User {
-fun main() : Unit {
+open fun main() : Unit {
 Library.ourOut?.print()
 }
 }

testData/ast/callChainExpression/file/libraryMethodCall.kt

@@ -1,15 +1,15 @@
 namespace {
 open class Library {
 class object {
-fun call() : Unit {
+open fun call() : Unit {
 }
-fun getString() : String? {
+open fun getString() : String? {
 return ""
 }
 }
 }
 open class User {
-fun main() : Unit {
+open fun main() : Unit {
 Library.call()
 Library.getString()?.isEmpty()
 }

testData/ast/callChainExpression/file/libraryMethodCallFromInstance.kt

@@ -1,13 +1,13 @@
 namespace {
 open class Library {
-fun call() : Unit {
+open fun call() : Unit {
 }
-fun getString() : String? {
+open fun getString() : String? {
 return ""
 }
 }
 open class User {
-fun main() : Unit {
+open fun main() : Unit {
 var lib : Library? = Library()
 lib?.call()
 lib?.getString()?.isEmpty()

testData/ast/callChainExpression/file/libraryStringFieldCall.kt

@@ -3,7 +3,7 @@ open class Library {
 public val myString : String?
 }
 open class User {
-fun main() : Unit {
+open fun main() : Unit {
 Library.myString?.isEmpty()
 }
 }

testData/ast/class/class/abstractClass.kt

@@ -1,6 +1,6 @@
 abstract open class A {
-abstract fun callme() : Unit
+abstract open fun callme() : Unit
-fun callmetoo() : Unit {
+open fun callmetoo() : Unit {
 print("This is a concrete method.")
 }
 }

testData/ast/class/class/abstractClassShape.kt

@@ -1,10 +1,10 @@
 abstract open class Shape {
 public var color : String?
-public fun setColor(c : String?) : Unit {
+open public fun setColor(c : String?) : Unit {
 color = c
 }
-public fun getColor() : String? {
+open public fun getColor() : String? {
 return color
 }
-abstract public fun area() : Double
+abstract open public fun area() : Double
 }

testData/ast/class/class/classWithEmptyMethods.kt

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

testData/ast/class/class/oneStaticFieldOneNonStatic.kt

@@ -2,7 +2,7 @@ class S {
 class object {
 var myI : Int = 10
 }
-fun sB() : Boolean {
+open fun sB() : Boolean {
 return true
 }
 }

testData/ast/class/class/oneStaticMethod.kt

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

testData/ast/class/class/oneStaticMethodOneNonStatic.kt

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

testData/ast/class/class/twoStaticMethod.kt

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

testData/ast/enum/class/fieldsWithPrimaryPrivateConstructor.kt

@@ -5,10 +5,10 @@ RED(23)
 YELLOW(24)
 BLUE(25)
 private var code : Int
-private (c : Int) {
+open private (c : Int) {
 code = c
 }
-public fun getCode() : Int {
+open public fun getCode() : Int {
 return code
 }
 }

testData/ast/function/file/override.kt

@@ -1,6 +1,6 @@
 namespace {
 open class A {
-fun a() : Unit {
+open fun a() : Unit {
 }
 }
 class B : A {

testData/ast/function/file/overrideAndOpen.jav

@@ -0,0 +1,11 @@
+class A {
+  void foo() {}
+}
+
+class B extends A {
+  void foo() {}
+}
+
+class C extends B {
+  void foo() {}
+}

testData/ast/function/file/overrideAndOpen.kt

@@ -0,0 +1,14 @@
+namespace {
+open class A {
+open fun foo() : Unit {
+}
+}
+open class B : A {
+override fun foo() : Unit {
+}
+}
+open class C : B {
+override fun foo() : Unit {
+}
+}
+}

testData/ast/function/method/abstractMethod.kt

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

testData/ast/function/method/classGenericParam.kt

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

testData/ast/function/method/emptyVoidMethod.kt

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

testData/ast/function/method/final.jav

@@ -0,0 +1 @@
+final String getString() { return ""; }

testData/ast/function/method/final.kt

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

testData/ast/function/method/internal.kt

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

testData/ast/function/method/methodClassType.kt

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

testData/ast/function/method/methodPrimitiveType.kt

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

testData/ast/function/method/methodPrimitiveType2.kt

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

testData/ast/function/method/methodWithReturnStatement.kt

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

testData/ast/function/method/open.jav

@@ -0,0 +1 @@
+String getString() { return ""; }

testData/ast/function/method/open.kt

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

testData/ast/function/method/ownGenericParam.kt

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

testData/ast/function/method/ownSeveralGenericParams.kt

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

testData/ast/function/method/private.kt

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

testData/ast/function/method/protected.kt

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

testData/ast/function/method/public.kt

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

testData/ast/inProjectionType/method/methodParams.kt

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

testData/ast/newClassExpression/file/newClassByFullName.kt

@@ -2,7 +2,7 @@ namespace org.test {
 open class Library {
 }
 open class User {
-fun main() : Unit {
+open fun main() : Unit {
 var lib : Library? = org.test.Library()
 }
 }

testData/ast/outProjectionType/method/methodParams.kt

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

testData/ast/starProjectionType/method/methodParams.kt

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

testData/ast/superExpression/class/classAextendsB.kt

@@ -1,5 +1,5 @@
 open class B {
-fun call() : Unit {
+open fun call() : Unit {
 }
 }
 open class A : B {

testData/ast/superExpression/file/classAdotSuperFoo.kt

@@ -1,10 +1,10 @@
 namespace {
 open class Base {
-fun foo() : Unit
+open fun foo() : Unit
 }
 open class A : Base {
 open class C {
-fun test() : Unit {
+open fun test() : Unit {
 super@A.foo()
 }
 }

testData/ast/thisExpression/file/classAdotThisFoo.kt

@@ -1,11 +1,11 @@
 namespace {
 open class Base {
-fun foo() : Unit {
+open fun foo() : Unit {
 }
 }
 open class A : Base {
 open class C {
-fun test() : Unit {
+open fun test() : Unit {
 this@A.foo()
 }
 }

testData/ast/trait/class/interfaceWithMethodDeclaration.kt

@@ -1,4 +1,4 @@
 trait INode {
-fun getTag() : Tag?
+open fun getTag() : Tag?
-fun toKotlin() : String?
+open fun toKotlin() : String?
 }

testData/ast/typeParameters/method/methodDoubleParametrizationWithTwoBounds.kt

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

testData/ast/varArg/method/ellipsisTypeSeveralParams.kt

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

testData/ast/varArg/method/ellipsisTypeSingleParams.kt

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