Get rid of compiler bugs workarounds

Author: qwwdfsad

binary-compatibility-validator/reference-public-api/kotlinx-coroutines-core.txt

@@ -1,10 +1,8 @@
 public abstract class kotlinx/coroutines/experimental/AbstractCoroutine : kotlin/coroutines/experimental/Continuation, kotlinx/coroutines/experimental/CoroutineScope, kotlinx/coroutines/experimental/Job {
 	public fun <init> (Lkotlin/coroutines/experimental/CoroutineContext;Z)V
 	public synthetic fun <init> (Lkotlin/coroutines/experimental/CoroutineContext;ZILkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public fun cancel (Ljava/lang/Throwable;)Z
 	public final fun getContext ()Lkotlin/coroutines/experimental/CoroutineContext;
 	public final fun getCoroutineContext ()Lkotlin/coroutines/experimental/CoroutineContext;
-	public fun invokeOnCompletion (ZZLkotlin/jvm/functions/Function1;)Lkotlinx/coroutines/experimental/DisposableHandle;
 	protected fun onCancellation (Ljava/lang/Throwable;)V
 	protected fun onCompleted (Ljava/lang/Object;)V
 	protected fun onCompletedExceptionally (Ljava/lang/Throwable;)V

common/kotlinx-coroutines-core-common/src/AbstractCoroutine.kt

@@ -152,17 +152,5 @@ public abstract class AbstractCoroutine<in T>(
         initParentJob()
         start(block, receiver, this)
     }
-
-    // todo: This workaround for KT-21968, should be removed in the future
-    override fun invokeOnCompletion(
-        onCancelling: Boolean,
-        invokeImmediately: Boolean,
-        handler: CompletionHandler
-    ): DisposableHandle =
-        super.invokeOnCompletion(onCancelling, invokeImmediately, handler)
-
-    // todo: This workaround for KT-21968, should be removed in the future
-    override fun cancel(cause: Throwable?) =
-        super.cancel(cause)
 }
 

common/kotlinx-coroutines-core-common/src/channels/Broadcast.kt

@@ -103,9 +103,6 @@ private open class BroadcastCoroutine<E>(
         if (!processed && cause != null)
             handleCoroutineException(context, cause)
     }
-
-    // Workaround for KT-23094
-    override suspend fun send(element: E) = _channel.send(element)
 }
 
 private class LazyBroadcastCoroutine<E>(

common/kotlinx-coroutines-core-common/src/channels/ChannelCoroutine.kt

@@ -12,15 +12,7 @@ internal open class ChannelCoroutine<E>(
     protected val _channel: Channel<E>,
     active: Boolean
 ) : AbstractCoroutine<Unit>(parentContext, active), Channel<E> by _channel {
-    val channel: Channel<E>
-        get() = this
-
-    // Workaround for KT-23094
-    override suspend fun receive(): E = _channel.receive()
-
-    override suspend fun send(element: E) = _channel.send(element)
-
-    override suspend fun receiveOrNull(): E? = _channel.receiveOrNull()
+    val channel: Channel<E> get() = this
 
     override fun cancel(cause: Throwable?): Boolean = super.cancel(cause)
 }

common/kotlinx-coroutines-core-common/test/channels/TestChannelKind.kt

@@ -55,9 +55,6 @@ private class ChannelViaBroadcast<E>(
     override val isClosedForReceive: Boolean get() = sub.isClosedForReceive
     override val isEmpty: Boolean get() = sub.isEmpty
 
-    // Workaround for KT-23094
-    override suspend fun send(element: E) = broadcast.send(element)
-
     override suspend fun receive(): E = sub.receive()
     override suspend fun receiveOrNull(): E? = sub.receiveOrNull()
     override fun poll(): E? = sub.poll()