Fix Kotlin warns, show disconnect reason on error.

retrixe · retrixe · commit afaaaa12347e · 2022-09-17T22:11:00.000+05:30
diff --git a/android/app/src/main/java/com/enderchat/modules/connection/ConnectionModule.kt b/android/app/src/main/java/com/enderchat/modules/connection/ConnectionModule.kt
@@ -18,7 +18,6 @@ import javax.crypto.Cipher
 import javax.crypto.spec.IvParameterSpec
 import javax.crypto.spec.SecretKeySpec
 import kotlin.concurrent.read
-import kotlin.concurrent.thread
 import kotlin.concurrent.write
 
 class ConnectionModule(reactContext: ReactApplicationContext)
@@ -291,9 +290,9 @@ class ConnectionModule(reactContext: ReactApplicationContext)
         // Remove upstream listeners, stop unnecessary background tasks
     }
 
-    private fun println(log: Any?) {
+    /* private fun println(log: Any?) {
         sendEvent(reactContext = reactApplicationContext, "ecm:log", Arguments.createMap().apply {
             putString("log", log.toString())
         })
-    }
+    } */
 }
diff --git a/src/minecraft/connection/javascript.ts b/src/minecraft/connection/javascript.ts
@@ -113,7 +113,10 @@ const initiateJavaScriptConnection = async (opts: ConnectionOptions) => {
     })
     socket.on('error', err => {
       if (!resolved) reject(err)
-      else conn.emit('error', err)
+      else {
+        conn.disconnectReason = err.message
+        conn.emit('error', err)
+      }
     })
     const lock = new Semaphore(1)
     socket.on('data', newData => {
diff --git a/src/minecraft/connection/native.ts b/src/minecraft/connection/native.ts
@@ -125,6 +125,7 @@ export class NativeServerConnection
     this.eventEmitter.addListener('ecm:error', (event: NativeErrorEvent) => {
       if (event.connectionId !== this.id) return
       console.error(event.stackTrace)
+      this.disconnectReason = event.message
       this.emit('error', new Error(event.message))
     })
     this.eventEmitter.addListener('ecm:close', (event: NativeEvent) => {
