Java to Kotlin converter: fixed bug

j2k/src/org/jetbrains/jet/j2k/TypeConverter.kt

@@ -69,6 +69,7 @@ class TypeConverter(val settings: ConverterSettings, val conversionScope: Conver
 
     private fun variableNullabilityNoCache(variable: PsiVariable): Nullability {
         if (variable is PsiEnumConstant) return Nullability.NotNull
+        if (variable.getType() is PsiPrimitiveType) return Nullability.NotNull
 
         var nullability = variable.nullabilityFromAnnotations()
 
@@ -154,6 +155,8 @@ class TypeConverter(val settings: ConverterSettings, val conversionScope: Conver
     }
 
     private fun methodNullabilityNoCache(method: PsiMethod): Nullability {
+        if (method.getReturnType() is PsiPrimitiveType) return Nullability.NotNull
+
         var nullability = method.nullabilityFromAnnotations()
 
         if (nullability == Nullability.Default) {

j2k/src/org/jetbrains/jet/j2k/ast/Types.kt

@@ -47,19 +47,9 @@ abstract class NotNullType() : Type() {
 abstract class Type() : Element() {
     abstract val isNullable: Boolean
 
-    open fun toNotNullType(): Type {
-        if (isNullable) {
-            throw UnsupportedOperationException("toNotNullType must be defined")
-        }
-        return this
-    }
+    open fun toNotNullType(): Type = this
 
-    open fun toNullableType(): Type {
-        if (!isNullable) {
-            throw UnsupportedOperationException("toNullableType must be defined")
-        }
-        return this
-    }
+    open fun toNullableType(): Type = this
 
     object Empty : NotNullType() {
         override fun generateCode(builder: CodeBuilder) {

j2k/tests/test/org/jetbrains/jet/j2k/test/JavaToKotlinConverterTestGenerated.java

@@ -2084,6 +2084,11 @@ public class JavaToKotlinConverterTestGenerated extends AbstractJavaToKotlinConv
             doTest("j2k/tests/testData/ast/nullability/MethodReturnsTernaryNull.java");
         }
         
+        @TestMetadata("NullableIntNoCrash.java")
+        public void testNullableIntNoCrash() throws Exception {
+            doTest("j2k/tests/testData/ast/nullability/NullableIntNoCrash.java");
+        }
+        
         @TestMetadata("NullableMethodDotAccess.java")
         public void testNullableMethodDotAccess() throws Exception {
             doTest("j2k/tests/testData/ast/nullability/NullableMethodDotAccess.java");

j2k/tests/testData/ast/nullability/NullableIntNoCrash.java

@@ -0,0 +1,8 @@
+import org.jetbrains.annotations.Nullable;
+
+//file
+class A {
+    int field = foo();
+
+    @Nullable int foo() { return 1; }
+}

j2k/tests/testData/ast/nullability/NullableIntNoCrash.kt

@@ -0,0 +1,7 @@
+class A() {
+    var field = foo()
+
+    fun foo(): Int {
+        return 1
+    }
+}