Is there any way NOT to have double underscore (__) before function names in swift?

[NikKovIos]

Hi)
Regarding this https://skie.touchlab.co/ there are examples with suspend functions:

But after implementing skie, we got __ before functions. I've readed https://skie.touchlab.co/features/suspend but still have a hope that it's not mandatory rule and we can safe original function names.

BTW __ in swift mean that it's a private API or internal var (objc rudiment) so it's not best naming.

[TadeasKriz]

Hi @NikKovIos, do you mean names when implementing an interface with suspending functions? Overriding Kotlin suspend functions from Swift is limited and doesn't propagate Swift's nor Kotlin's concurrency. That's true with or without SKIE.

__ is used specifically because it means "private API". We have to rename the original Kotlin declarations and hide them, so that Swift code can use the generated Swift async functions. That means Kotlin's suspend fun greet(): String becomes func __greet() async throws -> String and SKIE generates func greet() async throws -> String which properly works with Swift's concurrency.

[BroZZZ]

Hi @TadeasKriz, but what if we have something like this:

interface SomeInterface<P, R> {
    suspend fun run(param: P): R
}

abstract class SomeClassAbs<P, R : Any>() : SomeInterface<P, R> {
    open suspend operator fun invoke(param: P): R {
        return run(param)
    }
}

abstract class SomeClass :
    SomeClassAbs<SomeClassImpl.Params, SomeClassImpl.UseCaseResult>() {
    class Params()
    class UseCaseResult()
}

class SomeClassImpl : SomeClass() {
    override suspend fun run(params: Params):  SomeClassImpl.UseCaseResult {
        // some action and return 
    }
}

On IOS side, we cant call '.invoke' only '.__invoke'

let usecase: SomeClass
...
usecase.__invoke(params) - work's fine
usecase.invoke(params) - Value of type 'SomeClassImpl' has no member 'invoke'

I think generics are causing the issue here.

[TadeasKriz]

Yes that's a limitation of Objective-C and Swift described in SKIE docs: https://skie.touchlab.co/features/suspend#generic-classes

[NikKovIos]

@TadeasKriz Thx for answer!
Hope in future SKIE could separate generic functions and not generic. And for first not generate prefixes.☺️

[TadeasKriz]

The prefixes are there so you know you shouldn't be calling them. You want to use skie(x).invoke() for these, otherwise you're not getting cancellation and any-thread support.

But I think it'd be good for us to explore adding a deprecation notice on these methods that would point you to the skie(a).method() approach and/or SKIE documentation about it.

[NikKovIos]

Definetly. Without your words I would use __ methods. Now I would use skie() wrapper. Thanks again!