On Sun Jan 8 18:28:06 2023 +0000, Jinoh Kang wrote:
You should check for `impl->worker_thread` and `impl->worker_running` here. Due to how thread pools work, any user code can be executed between `session_StartAsync` and `session_start_async`.
Actually, I think you should move all the state checks from `StartAsync` to here. It doesn't hurt to fail later, as long as it fails correctly.