diff --git a/compiler/fir/checkers/src/org/jetbrains/kotlin/fir/analysis/checkers/extended/RedundantVisibilityModifierSyntaxChecker.kt b/compiler/fir/checkers/src/org/jetbrains/kotlin/fir/analysis/checkers/extended/RedundantVisibilityModifierSyntaxChecker.kt index 8299cbaef3f..487dc5a3088 100644 --- a/compiler/fir/checkers/src/org/jetbrains/kotlin/fir/analysis/checkers/extended/RedundantVisibilityModifierSyntaxChecker.kt +++ b/compiler/fir/checkers/src/org/jetbrains/kotlin/fir/analysis/checkers/extended/RedundantVisibilityModifierSyntaxChecker.kt @@ -59,20 +59,18 @@ object RedundantVisibilityModifierSyntaxChecker : FirDeclarationSyntaxChecker - val visibility = setter.implicitVisibility(it) - setterImplicitVisibility = visibility - checkElementAndReport(setter, visibility, property, it, reporter) - } + property.setter?.let { setter -> + val visibility = setter.implicitVisibility(context) + setterImplicitVisibility = visibility + checkElementAndReport(setter, visibility, property, context, reporter) + } - property.getter?.let { getter -> - checkElementAndReport(getter, property, it, reporter) - } + property.getter?.let { getter -> + checkElementAndReport(getter, property, context, reporter) + } - property.backingField?.let { field -> - checkElementAndReport(field, property, it, reporter) - } + property.backingField?.let { field -> + checkElementAndReport(field, property, context, reporter) } if (property.canMakeSetterMoreAccessible(setterImplicitVisibility)) { @@ -180,13 +178,10 @@ object RedundantVisibilityModifierSyntaxChecker : FirDeclarationSyntaxChecker= 2 - && context.containingDeclarations.asReversed()[1] is FirClass + && context.containingDeclarations.last() is FirClass && propertySymbol.isOverride -> findPropertyAccessorVisibility(this, context) - this is FirPropertyAccessor -> { - context.findClosest()?.visibility ?: Visibilities.DEFAULT_VISIBILITY - } + this is FirPropertyAccessor -> propertySymbol.visibility this is FirConstructor -> { val classSymbol = this.getContainingClassSymbol(context.session)