05nelsonm / kmp-tor

Kotlin Multiplatform Library for embedding Tor into your applications
Apache License 2.0
33 stars 5 forks source link

`AbstractTorServiceUI` should use immutable data class for state #503

Closed 05nelsonm closed 2 months ago

05nelsonm commented 2 months ago

Currently, separate variables are utilized for previous, displayed, and next state variables. Haven't had any issues because Android always utilized the main thread when binding to TorService, but the implementation should use a single class to hold and reference state within the function body where it is being worked with.

Part of #423