Stdlib warnings fixes

buildSrc/plugins/src/main/kotlin/org/jetbrains/kotlin/ExecutorService.kt

@@ -122,7 +122,7 @@ fun runProcess(executor: (Action<in ExecSpec>) -> ExecResult?,
     val stdOut = outStream.toString("UTF-8")
     val stdErr = errStream.toString("UTF-8")
 
-    return ProcessOutput(stdOut, stdErr, execResult!!.exitValue)
+    return ProcessOutput(stdOut, stdErr, execResult.exitValue)
 }
 
 fun runProcess(executor: (Action<in ExecSpec>) -> ExecResult?,

runtime/src/main/kotlin/kotlin/collections/Arrays.kt

@@ -15,7 +15,7 @@ import kotlin.util.sortArray
 /** Returns the array if it's not `null`, or an empty array otherwise. */
 public actual inline fun <reified T> Array<out T>?.orEmpty(): Array<out T> = this ?: emptyArray<T>()
 
-
+@Suppress("NOTHING_TO_INLINE")
 @PublishedApi internal inline fun checkCopyOfRangeArguments(fromIndex: Int, toIndex: Int, size: Int) {
     if (toIndex > size)
         throw IndexOutOfBoundsException("toIndex ($toIndex) is greater than size ($size).")
@@ -92,9 +92,8 @@ internal fun <T> Array<out T>.contentDeepHashCodeImpl(): Int {
     return result
 }
 
-internal actual fun <T> arrayOfNulls(reference: Array<T>, size: Int): Array<T> {
+@Suppress("UNCHECKED_CAST")
-    return (@Suppress("UNCHECKED_CAST")(arrayOfNulls<Any>(size)) as Array<T>)
+internal actual fun <T> arrayOfNulls(reference: Array<T>, size: Int): Array<T> = arrayOfNulls<Any>(size) as Array<T>
-}
 
 internal actual fun copyToArrayImpl(collection: Collection<*>): Array<Any?> {
     val array = arrayOfUninitializedElements<Any?>(collection.size)
@@ -105,6 +104,7 @@ internal actual fun copyToArrayImpl(collection: Collection<*>): Array<Any?> {
     return array
 }
 
+@Suppress("UNCHECKED_CAST")
 internal actual fun <T> copyToArrayImpl(collection: Collection<*>, array: Array<T>): Array<T> {
     if (array.size < collection.size)
         return copyToArrayImpl(collection) as Array<T>
@@ -115,7 +115,7 @@ internal actual fun <T> copyToArrayImpl(collection: Collection<*>, array: Array<
         array[index++] = iterator.next() as T
     }
     if (index < array.size) {
-        return (@Suppress("UNCHECKED_CAST")(array as Array<T>)).copyOf(index) as Array<T>
+        return array.copyOf(index) as Array<T>
     }
     return array
 }

runtime/src/main/kotlin/kotlin/collections/Collections.kt

@@ -11,11 +11,9 @@ import kotlin.random.*
 
 /** Copies typed varargs array to an array of objects */
 internal actual fun <T> Array<out T>.copyToArrayOfAny(isVarargs: Boolean): Array<out Any?> =
-        if (isVarargs)
+        // if the array came from varargs and already is array of Any, copying isn't required.
-            // if the array came from varargs and already is array of Any, copying isn't required.
+        if (isVarargs) this
-            @Suppress("UNCHECKED_CAST") (this as Array<out Any?>)
+        else this.copyOfUninitializedElements(this.size)
-        else
-            @Suppress("UNCHECKED_CAST") (this.copyOfUninitializedElements(this.size) as Array<out Any?>)
 
 
 /**

runtime/src/main/kotlin/kotlin/collections/Maps.kt

@@ -6,6 +6,7 @@
 package kotlin.collections
 
 // creates a singleton copy of map, if there is specialization available in target platform, otherwise returns itself
+@Suppress("NOTHING_TO_INLINE")
 internal inline actual fun <K, V> Map<K, V>.toSingletonMapOrSelf(): Map<K, V> = toSingletonMap()
 
 // creates a singleton copy of map

runtime/src/main/kotlin/kotlin/coroutines/DebugProbes.kt

@@ -54,6 +54,7 @@ internal fun <T> probeCoroutineCreated(completion: Continuation<T>): Continuatio
  * [BaseContinuationImpl] class, despite the fact that the declared type of [frame]
  * parameter in this function is `Continuation<*>`. See [probeCoroutineCreated] for details.
  */
+@Suppress("UNUSED_PARAMETER")
 @SinceKotlin("1.3")
 internal fun probeCoroutineResumed(frame: Continuation<*>) {
 }
@@ -68,6 +69,7 @@ internal fun probeCoroutineResumed(frame: Continuation<*>) {
  * [BaseContinuationImpl] class, despite the fact that the declared type of [frame]
  * parameter in this function is `Continuation<*>`. See [probeCoroutineCreated] for details.
  */
+@Suppress("UNUSED_PARAMETER")
 @SinceKotlin("1.3")
 internal fun probeCoroutineSuspended(frame: Continuation<*>) {
 }

runtime/src/main/kotlin/kotlin/coroutines/intrinsics/IntrinsicsNative.kt

@@ -71,6 +71,7 @@ private object CoroutineSuspendedMarker
  * state machine of the coroutine and may result in arbitrary behaviour or exception.
  */
 @SinceKotlin("1.3")
+@Suppress("UNCHECKED_CAST")
 public actual fun <T> (suspend () -> T).createCoroutineUnintercepted(
         completion: Continuation<T>
 ): Continuation<Unit> {
@@ -105,6 +106,7 @@ public actual fun <T> (suspend () -> T).createCoroutineUnintercepted(
  * state machine of the coroutine and may result in arbitrary behaviour or exception.
  */
 @SinceKotlin("1.3")
+@Suppress("UNCHECKED_CAST")
 public actual fun <R, T> (suspend R.() -> T).createCoroutineUnintercepted(
         receiver: R,
         completion: Continuation<T>
@@ -148,6 +150,7 @@ public actual fun <T> Continuation<T>.intercepted(): Continuation<T> =
  * The instance of [BaseContinuationImpl] is passed to the [block] so that it can be passed to the corresponding invocation.
  */
 @SinceKotlin("1.3")
+@Suppress("UNCHECKED_CAST")
 private inline fun <T> createCoroutineFromSuspendFunction(
         completion: Continuation<T>,
         crossinline block: (Continuation<T>) -> Any?

runtime/src/main/kotlin/kotlin/native/internal/KPropertyImpl.kt

@@ -34,8 +34,8 @@ open class KProperty0Impl<out R>(override val name: String, override val returnT
 
 @FixmeReflection
 open class KProperty1Impl<T, out R>(override val name: String, override val returnType: KType, val getter: (T) -> R): KProperty1<T, R> {
-    override fun get(p1: T): R {
+    override fun get(receiver: T): R {
-        return getter(p1)
+        return getter(receiver)
     }
     override fun invoke(p1: T): R {
         return getter(p1)
@@ -58,8 +58,8 @@ open class KProperty1Impl<T, out R>(override val name: String, override val retu
 
 @FixmeReflection
 open class KProperty2Impl<T1, T2, out R>(override val name: String, override val returnType: KType, val getter: (T1, T2) -> R): KProperty2<T1, T2, R> {
-    override fun get(p1: T1, p2: T2): R {
+    override fun get(receiver1: T1, receiver2: T2): R {
-        return getter(p1, p2)
+        return getter(receiver1, receiver2)
     }
     override fun invoke(p1: T1, p2: T2): R {
         return getter(p1, p2)

runtime/src/main/kotlin/kotlin/native/ref/Weak.kt

@@ -35,6 +35,7 @@ public class WeakReference<T : Any> {
     /**
      * Returns either reference to an object or null, if it was collected.
      */
+    @Suppress("UNCHECKED_CAST")
     public fun get(): T? = pointer?.get() as T?
 }
 

runtime/src/main/kotlin/kotlin/text/regex/sets/AbstractSet.kt

@@ -46,6 +46,7 @@ internal abstract class AbstractSet(val type: Int = 0) {
         val dummyNext = object : AbstractSet() {
             override var next: AbstractSet
                 get() = throw AssertionError("This method is not expected to be called.")
+                @Suppress("UNUSED_PARAMETER")
                 set(value) {}
             override fun matches(startIndex: Int, testString: CharSequence, matchResult: MatchResultImpl) =
                 throw AssertionError("This method is not expected to be called.")