From 7a9021aa6bb2d6dbef55835042c027a7d200fe0d Mon Sep 17 00:00:00 2001 From: Roman Golyshev Date: Tue, 9 May 2023 13:43:05 +0200 Subject: [PATCH] [AA FIR] KT-58503 Do not use `resolvedStatus` for `visibility` yet It's done to avoid lazy resolve contract violation errors. See KT-58572 --- .../api/fir/symbols/KtFirNamedClassOrObjectSymbol.kt | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/analysis/analysis-api-fir/src/org/jetbrains/kotlin/analysis/api/fir/symbols/KtFirNamedClassOrObjectSymbol.kt b/analysis/analysis-api-fir/src/org/jetbrains/kotlin/analysis/api/fir/symbols/KtFirNamedClassOrObjectSymbol.kt index 290b95f8d12..d9114285a6e 100644 --- a/analysis/analysis-api-fir/src/org/jetbrains/kotlin/analysis/api/fir/symbols/KtFirNamedClassOrObjectSymbol.kt +++ b/analysis/analysis-api-fir/src/org/jetbrains/kotlin/analysis/api/fir/symbols/KtFirNamedClassOrObjectSymbol.kt @@ -48,8 +48,9 @@ internal class KtFirNamedClassOrObjectSymbol( override val visibility: Visibility get() = withValidityAssertion { - when (val possiblyRawVisibility = firSymbol.visibility) { - Visibilities.Unknown -> if (firSymbol.isLocal) Visibilities.Local else Visibilities.Public + // TODO: We should use resolvedStatus, because it can be altered by status-transforming compiler plugins. See KT-58572 + when (val possiblyRawVisibility = firSymbol.fir.visibility) { + Visibilities.Unknown -> if (firSymbol.fir.isLocal) Visibilities.Local else Visibilities.Public else -> possiblyRawVisibility } }