Merge pull request #205 from arkivanov/enable-JS-IR

Enable JS IR

Author: arkivanov

buildSrc/buildSrc/src/main/kotlin/dependencies/Deps.kt

@@ -72,7 +72,7 @@ object Deps {
 
     object Badoo {
         object Reaktive : Group(name = "com.badoo.reaktive") {
-            private const val version = "1.1.20"
+            private const val version = "1.1.21"
 
             object Reaktive : Dependency(group = Badoo.Reaktive, name = "reaktive", version = version)
             object ReaktiveAnnotations : Dependency(group = Badoo.Reaktive, name = "reaktive-annotations", version = version)

gradle.properties

@@ -1,5 +1,6 @@
 kotlin.code.style=official
 kotlin.native.disableCompilerDaemon=true
+kotlin.js.compiler=both
 org.gradle.jvmargs=-Xmx2g -XX:MaxMetaspaceSize=512m -Dfile.encoding=UTF-8
 org.gradle.parallel=true
 org.gradle.caching=true

rx/src/commonMain/kotlin/com/arkivanov/mvikotlin/rx/ObserverBuilder.kt

@@ -1,4 +1,19 @@
 package com.arkivanov.mvikotlin.rx
 
-// Expect/actual workaround for https://github.com/arkivanov/MVIKotlin/issues/145
-expect fun <T> observer(onComplete: () -> Unit = {}, onNext: (T) -> Unit = {}): Observer<T>
+// Not inlined until https://youtrack.jetbrains.com/issue/KT-45866.
+// Also check https://github.com/arkivanov/MVIKotlin/issues/145 before making it inlined.
+fun <T> observer(onComplete: (() -> Unit)? = null, onNext: ((T) -> Unit)? = null): Observer<T> =
+    ObserverImpl(onComplete, onNext)
+
+private class ObserverImpl<in T>(
+    private val onComplete: (() -> Unit)? = null,
+    private val onNext: ((T) -> Unit)? = null
+) : Observer<T> {
+    override fun onNext(value: T) {
+        onNext?.invoke(value)
+    }
+
+    override fun onComplete() {
+        onComplete?.invoke()
+    }
+}

rx/src/jsMain/kotlin/com/arkivanov/mvikotlin/rx/ObserverBuilder.kt

@@ -3,7 +3,7 @@ plugins {
 }
 
 kotlin {
-    js {
+    js(LEGACY) {
         useCommonJs()
         browser()
     }