Fix compiler warnings

Fixes Kotlin#348

Author: qwwdfsad

common/kotlinx-coroutines-core-common/src/main/kotlin/kotlinx/coroutines/experimental/CoroutineContext.common.kt

@@ -18,7 +18,6 @@ package kotlinx.coroutines.experimental
 
 import kotlin.coroutines.experimental.*
 
-@Suppress("EXPECTED_DECLARATION_WITH_DEFAULT_PARAMETER")
 public expect fun newCoroutineContext(context: CoroutineContext, parent: Job? = null): CoroutineContext
 
 @Suppress("PropertyName")

common/kotlinx-coroutines-core-common/src/test/kotlin/kotlinx/coroutines/experimental/TestBase.common.kt

@@ -20,13 +20,11 @@ public expect open class TestBase constructor() {
     public val isStressTest: Boolean
     public val stressTestMultiplier: Int
 
-    @Suppress("EXPECTED_DECLARATION_WITH_DEFAULT_PARAMETER")
     public fun error(message: Any, cause: Throwable? = null): Nothing
     public fun expect(index: Int)
     public fun expectUnreached()
     public fun finish(index: Int)
 
-    @Suppress("EXPECTED_DECLARATION_WITH_DEFAULT_PARAMETER")
     public fun runTest(
         expected: ((Throwable) -> Boolean)? = null,
         unhandled: List<(Throwable) -> Boolean> = emptyList(),

common/kotlinx-coroutines-core-common/src/test/kotlin/kotlinx/coroutines/experimental/WithTimeoutOrNullTest.kt

@@ -177,7 +177,7 @@ class WithTimeoutOrNullTest : TestBase() {
         expected = { it is TestException }
     ) {
         expect(1)
-        val result = withTimeoutOrNull(100) {
+        withTimeoutOrNull(100) {
             expect(2)
             try {
                 delay(1000)

common/kotlinx-coroutines-core-common/src/test/kotlin/kotlinx/coroutines/experimental/WithTimeoutTest.kt

@@ -147,7 +147,7 @@ class WithTimeoutTest : TestBase() {
         expected = { it is CancellationException }
     ) {
         expect(1)
-        val result = withTimeout(100) {
+        withTimeout(100) {
             expect(2)
             try {
                 delay(1000)

core/kotlinx-coroutines-core/src/main/kotlin/kotlinx/coroutines/experimental/CommonPool.kt

@@ -18,8 +18,8 @@ package kotlinx.coroutines.experimental
 
 import kotlinx.coroutines.experimental.timeunit.TimeUnit
 import java.util.concurrent.*
-import java.util.concurrent.atomic.AtomicInteger
-import kotlin.coroutines.experimental.CoroutineContext
+import java.util.concurrent.atomic.*
+import kotlin.coroutines.experimental.*
 
 /**
  * Represents common pool of shared threads as coroutine dispatcher for compute-intensive tasks.

core/kotlinx-coroutines-core/src/test/kotlin/kotlinx/coroutines/experimental/AsyncJvmTest.kt

@@ -24,7 +24,9 @@ class AsyncJvmTest : TestBase() {
     @Test
     fun testAsyncWithFinally() = runTest {
         expect(1)
-        val d = async<String>(coroutineContext) {
+
+        @Suppress("UNREACHABLE_CODE")
+        val d = async(coroutineContext) {
             expect(3)
             try {
                 yield() // to main, will cancel

core/kotlinx-coroutines-core/src/test/kotlin/kotlinx/coroutines/experimental/TestBaseTest.kt

@@ -22,7 +22,7 @@ class TestBaseTest : TestBase() {
     @Test
     fun testThreadsShutdown() {
         val SHUTDOWN_TIMEOUT = 1_000L
-        repeat(1000 * stressTestMultiplier) { i ->
+        repeat(1000 * stressTestMultiplier) { _ ->
             CommonPool.usePrivatePool()
             val threadsBefore = currentThreads()
             runBlocking {

core/kotlinx-coroutines-core/src/test/kotlin/kotlinx/coroutines/experimental/channels/ActorTest.kt

@@ -106,7 +106,7 @@ class ActorTest(private val capacity: Int) : TestBase() {
             expect(2)
             require(element!! == 42)
             try {
-                val next = channel.receiveOrNull()
+                channel.receiveOrNull()
             } catch (e: IOException) {
                 expect(3)
             }

core/kotlinx-coroutines-core/src/test/kotlin/kotlinx/coroutines/experimental/channels/BroadcastChannelMultiReceiveStressTest.kt

@@ -91,7 +91,7 @@ class BroadcastChannelMultiReceiveStressTest(
             printProgress()
         }
         // wait
-        repeat(nSeconds) { sec ->
+        repeat(nSeconds) { _ ->
             delay(1000)
             printProgress()
         }

core/kotlinx-coroutines-core/src/test/kotlin/kotlinx/coroutines/experimental/internal/LockFreeListLinearizabilityTest.kt

@@ -53,7 +53,7 @@ class LockFreeListLinearizabilityTest : TestBase() {
     @Operation
     fun removeFirstOrPeekIfNotSame(@Param(name = "value") value: Int): Int? {
         val node = q.removeFirstIfIsInstanceOfOrPeekIf<Node> { !it.isSame(value) } ?: return null
-        return (node as Node).value
+        return node.value
     }
 
     fun Any.isSame(value: Int) = this is Node && this.value == value

core/kotlinx-coroutines-io/src/test/kotlin/kotlinx/coroutines/experimental/io/ByteBufferChannelTest.kt

@@ -819,7 +819,7 @@ class ByteBufferChannelTest : TestBase() {
         return launch(context = DefaultDispatcher + CoroutineName(name), parent = coroutineContext[Job]) {
             block()
         }.apply {
-            invokeOnCompletion(true) { t ->
+            invokeOnCompletion( onCancelling = true) { t ->
                 if (t != null) ch.cancel(t)
             }
         }

integration/kotlinx-coroutines-jdk8/src/main/kotlin/kotlinx/coroutines/experimental/future/Future.kt

@@ -166,7 +166,7 @@ public suspend fun <T> CompletionStage<T>.await(): T {
     // fast path when CompletableFuture is already done (does not suspend)
     if (this is Future<*> && isDone()) {
         try {
-            @Suppress("UNCHECKED")
+            @Suppress("UNCHECKED_CAST")
             return get() as T
         } catch (e: ExecutionException) {
             throw e.cause ?: e // unwrap original cause from ExecutionException