From 9a151e1422516c1e41e85a5da67662f0213624ef Mon Sep 17 00:00:00 2001 From: pyos Date: Thu, 13 Oct 2022 12:44:56 +0200 Subject: [PATCH] Minor: remove unused FirOverrideService methods --- .../kotlin/fir/scopes/FirOverrideService.kt | 50 ------------------- 1 file changed, 50 deletions(-) diff --git a/compiler/fir/providers/src/org/jetbrains/kotlin/fir/scopes/FirOverrideService.kt b/compiler/fir/providers/src/org/jetbrains/kotlin/fir/scopes/FirOverrideService.kt index 6c8f039802c..416637fc997 100644 --- a/compiler/fir/providers/src/org/jetbrains/kotlin/fir/scopes/FirOverrideService.kt +++ b/compiler/fir/providers/src/org/jetbrains/kotlin/fir/scopes/FirOverrideService.kt @@ -19,46 +19,9 @@ import org.jetbrains.kotlin.fir.types.ConeFlexibleType import org.jetbrains.kotlin.fir.types.ConeKotlinType import org.jetbrains.kotlin.fir.types.typeContext import org.jetbrains.kotlin.types.AbstractTypeChecker -import org.jetbrains.kotlin.utils.SmartSet import java.util.* class FirOverrideService(val session: FirSession) : FirSessionComponent { - fun > selectMostSpecificInEachOverridableGroup( - members: Collection>, - overrideChecker: FirOverrideChecker, - returnTypeCalculator: ReturnTypeCalculator - ): Collection> { - if (members.size <= 1) return members - val queue = LinkedList(members) - val result = SmartSet.create>() - - while (queue.isNotEmpty()) { - val nextHandle = queue.first() - - val conflictedHandles = SmartSet.create>() - - val overridableGroup = extractBothWaysOverridable(nextHandle, queue, overrideChecker) - - if (overridableGroup.size == 1 && conflictedHandles.isEmpty()) { - result.add(overridableGroup.single()) - continue - } - - val mostSpecific = selectMostSpecificMember(overridableGroup, returnTypeCalculator) - - overridableGroup.filterNotTo(conflictedHandles) { - isMoreSpecificOrEqual(mostSpecific, it, returnTypeCalculator) - } - - if (conflictedHandles.isNotEmpty()) { - result.addAll(conflictedHandles) - } - - result.add(mostSpecific) - } - return result - } - fun > createOverridableGroups( members: Collection>, overrideChecker: FirOverrideChecker @@ -133,11 +96,6 @@ class FirOverrideService(val session: FirSession) : FirSessionComponent { return maximums.map { it.memberWithBaseScope } } - fun > selectMostSpecificMember( - overridables: List>, - returnTypeCalculator: ReturnTypeCalculator - ): MemberWithBaseScope = selectMostSpecificMembers(overridables, returnTypeCalculator).first() - private fun MutableList.removeFlagged(flags: BooleanArray) { var dest = 0 for (i in flags.indices) { @@ -150,14 +108,6 @@ class FirOverrideService(val session: FirSession) : FirSessionComponent { } } - private fun isMoreSpecificOrEqual( - a: MemberWithBaseScope<*>, - b: MemberWithBaseScope<*>, - returnTypeCalculator: ReturnTypeCalculator - ) = MemberWithBaseScopeAndReturnType(a, returnTypeCalculator).compareTo(MemberWithBaseScopeAndReturnType(b, returnTypeCalculator)).let { - it != null && it >= 0 - } - private class MemberWithBaseScopeAndReturnType>( val memberWithBaseScope: MemberWithBaseScope, returnTypeCalculator: ReturnTypeCalculator