kpgalligan
04/30/2020, 1:02 PMphldavies
04/30/2020, 1:09 PMnetwork
and removed the withContext
wrapper - seems to work finecoroutineScope
too and just leave the MainThread guard in place - will the coroutineScope
have any impact on cancellation?kpgalligan
04/30/2020, 1:15 PMrusshwolf
04/30/2020, 1:17 PMcoroutineScope
was to try and play nice with typical coroutine semantics if you wanted to add a launch internally or something. But realistically that probably just makes it easier to do something that won't work.phldavies
04/30/2020, 2:48 PMcoroutineScope
in the network
function on Android would at least keep the coroutine semantics better aligned between the two and doesn’t exhibit the deadlock I was seeing on Android with both the coroutineScope
and the withContext(childContext())