K2: Simplify condition in RedundantVisibilityModifierSyntaxChecker
element !is FirMemberDeclaration means that `isAccessorWithSameVisibility == false` because `FirPropertyAccessor <: FirMemberDeclaration` Also, remove a function became unused
This commit is contained in:
committed by
Space Team
parent
de444542d0
commit
2cd6b8253f
+1
-7
@@ -117,10 +117,7 @@ object RedundantVisibilityModifierSyntaxChecker : FirDeclarationSyntaxChecker<Fi
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
val isAccessorWithSameVisibility = element is FirPropertyAccessor
|
if (element !is FirMemberDeclaration) {
|
||||||
&& element.visibility == context.containingPropertyVisibility
|
|
||||||
|
|
||||||
if (element !is FirMemberDeclaration && !isAccessorWithSameVisibility) {
|
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -281,7 +278,4 @@ object RedundantVisibilityModifierSyntaxChecker : FirDeclarationSyntaxChecker<Fi
|
|||||||
scope.processOverriddenFunctions(function.symbol, it)
|
scope.processOverriddenFunctions(function.symbol, it)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private val CheckerContext.containingPropertyVisibility
|
|
||||||
get() = (this.containingDeclarations.last() as? FirProperty)?.visibility
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user