Skip to content

Commit 89df171

Browse files
refactor: destructure access to PartitionResult.
1 parent 20647a3 commit 89df171

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

src/main/kotlin/com/autonomousapps/internal/binary/BinaryCompatibilityChecker.kt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import com.autonomousapps.model.internal.BinaryClassCapability
1212
import com.autonomousapps.model.internal.intermediates.consumer.MemberAccess
1313
import com.autonomousapps.model.internal.intermediates.producer.BinaryClass
1414
import com.autonomousapps.visitor.GraphViewVisitor
15-
import kotlin.collections.forEach
1615

1716
internal class BinaryCompatibilityChecker(
1817
private val coordinates: Coordinates,
@@ -46,11 +45,9 @@ internal class BinaryCompatibilityChecker(
4645
val relevantMemberAccesses = context.project.memberAccesses
4746
.filterToOrderedSet { access -> access.owner in relevantDuplicateClassNames }
4847

49-
val partitionResult = relevantMemberAccesses.mapToSet { access ->
48+
val (matchingBinaryClasses, nonMatchingBinaryClasses) = relevantMemberAccesses.mapToSet { access ->
5049
binaryClassCapability.findMatchingClasses(access)
5150
}.reduce()
52-
val matchingBinaryClasses = partitionResult.matchingClasses
53-
val nonMatchingBinaryClasses = partitionResult.nonMatchingClasses
5451

5552
// There must be a compatible BinaryClass.<field|method> for each MemberAccess for the usage to be binary-compatible
5653
val isBinaryCompatible = relevantMemberAccesses.all { access ->

src/main/kotlin/com/autonomousapps/model/internal/intermediates/producer/BinaryClass.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ internal data class BinaryClass(
3030
.compare(this, other)
3131
}
3232

33-
// TODO(tsr): currently unused. Delete?
3433
internal class Builder(
3534
val className: String,
3635
val superClassName: String?,

0 commit comments

Comments
 (0)