FIR: make Java default constructor visibility same with class visibility

This commit is contained in:
Mikhail Glukhikh
2019-07-10 16:56:08 +03:00
parent 14bbbb5bf5
commit 2ceffa241b
9 changed files with 9 additions and 10 deletions
@@ -8,7 +8,6 @@ package org.jetbrains.kotlin.fir.java
import com.intellij.openapi.project.Project
import com.intellij.psi.search.GlobalSearchScope
import org.jetbrains.kotlin.descriptors.ClassKind
import org.jetbrains.kotlin.descriptors.Visibilities
import org.jetbrains.kotlin.descriptors.Visibility
import org.jetbrains.kotlin.fir.FirSession
import org.jetbrains.kotlin.fir.declarations.FirRegularClass
@@ -211,7 +210,7 @@ class JavaSymbolProvider(
val javaClassDeclaredConstructors = javaClass.constructors
val constructorId = CallableId(classId.packageFqName, classId.relativeClassName, classId.shortClassName)
fun addJavaConstructor(visibility: Visibility = Visibilities.PUBLIC): FirJavaConstructor {
fun addJavaConstructor(visibility: Visibility = this.visibility): FirJavaConstructor {
val constructorSymbol = FirConstructorSymbol(constructorId)
val classTypeParameters = javaClass.typeParameters.convertTypeParameters(javaTypeParameterStack)
val firJavaConstructor = FirJavaConstructor(
@@ -7,7 +7,7 @@ public/*package*/ open class A : R|kotlin/Any| {
public open operator fun bam(@R|kotlin/annotations/jvm/internal/DefaultValue|(String(NOT_ENTRY_EITHER)) arg: R|ft<Mixed, Mixed?>|! = R|/Mixed.NOT_ENTRY_EITHER|): R|ft<Mixed, Mixed?>|!
public constructor(): R|A|
public/*package*/ constructor(): R|A|
}
public final enum class Mixed : R|kotlin/Enum<Mixed>| {
@@ -9,6 +9,6 @@ public/*package*/ open class A : R|kotlin/Any| {
public open operator fun wrong(@R|kotlin/annotations/jvm/internal/DefaultValue|(String(hello)) i: R|ft<kotlin/Int, kotlin/Int?>|!): R|kotlin/Unit|
public constructor(): R|A|
public/*package*/ constructor(): R|A|
}
@@ -5,6 +5,6 @@ public/*package*/ open class A : R|kotlin/Any| {
public open operator fun numberName(@R|kotlin/annotations/jvm/internal/ParameterName|(Int(42)) first: R|ft<kotlin/String, kotlin/String?>|!): R|kotlin/Unit|
public constructor(): R|A|
public/*package*/ constructor(): R|A|
}
@@ -1,6 +1,6 @@
public/*package*/ open class A : R|kotlin/Any| {
public open static operator fun withDefault(@R|kotlin/annotations/jvm/internal/DefaultValue|(String(OK)) arg: R|ft<kotlin/String, kotlin/String?>|! = String(OK)): R|ft<kotlin/String, kotlin/String?>|!
public constructor(): R|A|
public/*package*/ constructor(): R|A|
}