2) Is it a compiler bug that suspending function references are no longer accepted if I remove the s...

# coroutines

[uli]

2) Is it a compiler bug that suspending function references are no longer accepted if I remove the suspend modifier?

[Dico]

Example?

[uli]

class X {

    suspend fun suspending() {}

    suspend fun test1() {
        // Error goes away if we add suspend modifier to logError an it's parameter block
        // But then IDE complains about redundant suspend modifier
        logError(::suspending)
    }

    suspend fun test2() {
        logError { suspending() }
    }

    inline fun logError(block: () -> Unit) {
        try {
            block()
        } catch (e: Exception) {
            //Log Exception
        }
    }
}

[Vsevolod Tolstopyatov [JB]]

It is both bug in the compiler and in the inspection 🙂
https://youtrack.jetbrains.com/issue/KT-30536

[Vsevolod Tolstopyatov [JB]]

https://youtrack.jetbrains.com/issue/KT-34039 filed inspection bug

[uli]

@Vsevolod Tolstopyatov [JB] thanks for opening the ticket and especially for pointing out the name of the inspection 🙂

[uli]

And now I get a wrong bytecode generated error for this:

class X {

    suspend fun suspending() {}

    suspend fun test() {
        logError(::suspending)
    }

    suspend inline fun runBlock(block: suspend () -> Unit) {
        logError(block)
    }

    suspend inline fun logError(block: suspend () -> Unit) {
        try {
            block()
        } catch (e: Exception) {
            //Log Exception
        }
    }
}

[Ilmir Usmanov [JB]]

@uli Can you, please, open a separate ticket for this case? I'll take a look.

[uli]

https://youtrack.jetbrains.com/issue/KT-34041