Skip to content

Weak Reference Fix#900

Closed
Daeda88 wants to merge 9 commits intodevelopfrom
feature/weak-reference-as-cinterop
Closed

Weak Reference Fix#900
Daeda88 wants to merge 9 commits intodevelopfrom
feature/weak-reference-as-cinterop

Conversation

@Daeda88
Copy link
Contributor

@Daeda88 Daeda88 commented Aug 18, 2025

Wraps ObjC Weak references in Swift classes so there wont be a mismatch between the Kotlin and ObjC memory model causing a deadlock

project.configure(iosTargets) {
compilations.getByName("main").cinterops.let { mainInterops ->
appleInterop.main.forEach { it.execute(mainInterops) }
val swiftSourceDir = project.file("src/iosMain/swift")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we ever move this to a seperate plugin consider using sourceset

Copy link
Collaborator

@thoutbeckers thoutbeckers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some small clean up things, otherwise fine

@ci-splendo
Copy link
Contributor

Code coverage

Total Project Coverage 35.51%

@thoutbeckers
Copy link
Collaborator

superseded by #903

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants