From 2cd6b8253f29aee4dc77d4f92f2e3437aede6ee4 Mon Sep 17 00:00:00 2001
From: "Denis.Zharkov" <denis.zharkov@jetbrains.com>
Date: Mon, 6 Feb 2023 18:32:41 +0100
Subject: [PATCH] K2: Simplify condition in
 RedundantVisibilityModifierSyntaxChecker

element !is FirMemberDeclaration means that
`isAccessorWithSameVisibility == false` because
`FirPropertyAccessor <: FirMemberDeclaration`

Also, remove a function became unused
---
 .../extended/RedundantVisibilityModifierSyntaxChecker.kt  | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

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 b88a29b9f97..8299cbaef3f 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
@@ -117,10 +117,7 @@ object RedundantVisibilityModifierSyntaxChecker : FirDeclarationSyntaxChecker<Fi
             return
         }
 
-        val isAccessorWithSameVisibility = element is FirPropertyAccessor
-                && element.visibility == context.containingPropertyVisibility
-
-        if (element !is FirMemberDeclaration && !isAccessorWithSameVisibility) {
+        if (element !is FirMemberDeclaration) {
             return
         }
 
@@ -281,7 +278,4 @@ object RedundantVisibilityModifierSyntaxChecker : FirDeclarationSyntaxChecker<Fi
             scope.processOverriddenFunctions(function.symbol, it)
         }
     }
-
-    private val CheckerContext.containingPropertyVisibility
-        get() = (this.containingDeclarations.last() as? FirProperty)?.visibility
 }