Java to Kotlin conversion: artificial constructor (when no primary detected) is generated private

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

@@ -160,7 +160,7 @@ public class Converter(val project: Project, val settings: ConverterSettings) {
 
                     //TODO: comments?
                     val parameters = finalOrWithEmptyInitializerFields.map { Parameter(Identifier("_" + it.identifier.name), it.`type`, Parameter.VarValModifier.None, listOf()) }
-                    classBodyElements.add(PrimaryConstructor(this, MemberComments.Empty, setOf(),
+                    classBodyElements.add(PrimaryConstructor(this, MemberComments.Empty, setOf(Modifier.PRIVATE),
                                                              ParameterList(parameters),
                                                              Block(createInitStatementsFromFields(finalOrWithEmptyInitializerFields))))
                 }

j2k/tests/testData/ast/class/kt-639.ide.kt

@@ -2,7 +2,7 @@ package demo
 
 import java.util.HashMap
 
-class Test() {
+class Test private() {
     class object {
         fun init(): Test {
             val __ = Test()

j2k/tests/testData/ast/class/kt-639.kt

@@ -2,7 +2,7 @@ package demo
 
 import java.util.HashMap
 
-open class Test() {
+open class Test private() {
     class object {
         open fun init(): Test {
             val __ = Test()

j2k/tests/testData/ast/constructors/genericIdentifier.ide.kt

@@ -1,4 +1,4 @@
-public class Identifier<T>(_myName: T, _myHasDollar: Boolean) {
+public class Identifier<T> private(_myName: T, _myHasDollar: Boolean) {
     private val myName: T
     private var myHasDollar: Boolean = false
     private var myNullable: Boolean = true

j2k/tests/testData/ast/constructors/genericIdentifier.kt

@@ -1,4 +1,4 @@
-public open class Identifier<T>(_myName: T?, _myHasDollar: Boolean) {
+public open class Identifier<T> private(_myName: T?, _myHasDollar: Boolean) {
     private val myName: T?
     private var myHasDollar: Boolean = false
     private var myNullable: Boolean = true

j2k/tests/testData/ast/constructors/identifier.ide.kt

@@ -1,4 +1,4 @@
-public class Identifier(_myName: String, _myHasDollar: Boolean) {
+public class Identifier private(_myName: String, _myHasDollar: Boolean) {
     private val myName: String
     private var myHasDollar: Boolean = false
     private var myNullable: Boolean = true

j2k/tests/testData/ast/constructors/identifier.kt

@@ -1,4 +1,4 @@
-public open class Identifier(_myName: String?, _myHasDollar: Boolean) {
+public open class Identifier private(_myName: String?, _myHasDollar: Boolean) {
     private val myName: String?
     private var myHasDollar: Boolean = false
     private var myNullable: Boolean = true

j2k/tests/testData/ast/constructors/withManyDefaultParams.ide.kt

@@ -1,4 +1,4 @@
-public class Test(_myName: String, _a: Boolean, _b: Double, _c: Float, _d: Long, _e: Int, _f: Short, _g: Char) {
+public class Test private(_myName: String, _a: Boolean, _b: Double, _c: Float, _d: Long, _e: Int, _f: Short, _g: Char) {
     private val myName: String
     private var a: Boolean = false
     private var b: Double = 0.toDouble()

j2k/tests/testData/ast/constructors/withManyDefaultParams.kt

@@ -1,4 +1,4 @@
-public open class Test(_myName: String?, _a: Boolean, _b: Double, _c: Float, _d: Long, _e: Int, _f: Short, _g: Char) {
+public open class Test private(_myName: String?, _a: Boolean, _b: Double, _c: Float, _d: Long, _e: Int, _f: Short, _g: Char) {
     private val myName: String?
     private var a: Boolean = false
     private var b: Double = 0.toDouble()

j2k/tests/testData/ast/issues/kt-638.ide.kt

@@ -1,4 +1,4 @@
-public class Identifier<T>(_myName: T, _myHasDollar: Boolean) {
+public class Identifier<T> private(_myName: T, _myHasDollar: Boolean) {
     private val myName: T
     private var myHasDollar: Boolean = false
     private var myNullable: Boolean = true

j2k/tests/testData/ast/issues/kt-638.kt

@@ -1,4 +1,4 @@
-public open class Identifier<T>(_myName: T?, _myHasDollar: Boolean) {
+public open class Identifier<T> private(_myName: T?, _myHasDollar: Boolean) {
     private val myName: T?
     private var myHasDollar: Boolean = false
     private var myNullable: Boolean = true