From a6a39d1234edb3014bc1aebe1fa013555e477f64 Mon Sep 17 00:00:00 2001 From: pierre Date: Tue, 30 May 2023 13:58:30 +0200 Subject: [PATCH] chore(nc-native-android): remove outdated todo --- .../app/src/main/java/net/nymtech/nyms5/MainViewModel.kt | 6 ------ 1 file changed, 6 deletions(-) diff --git a/nym-connect/native/android/app/src/main/java/net/nymtech/nyms5/MainViewModel.kt b/nym-connect/native/android/app/src/main/java/net/nymtech/nyms5/MainViewModel.kt index 36f4858dc7..a795bfdfe4 100644 --- a/nym-connect/native/android/app/src/main/java/net/nymtech/nyms5/MainViewModel.kt +++ b/nym-connect/native/android/app/src/main/java/net/nymtech/nyms5/MainViewModel.kt @@ -39,12 +39,6 @@ class MainViewModel( init { Log.d(tag, "____init") - // TODO ⚠ In some circumstances this `init` block can be run multiple - // time which means the below observer will be registered more than once - // This can leads to multiple sequential calls to `stopClient` - // → nym_socks5_listener panics when this happens, crashing the - // entire App - // When the work is cancelled "externally" ie. when the user tap the // "Stop" action on the notification, or when the app is intentionally // killed the underlying proxy client keeps running in background