From 6f5c7b2452365593ddd33d620c631c40b5c2beaa Mon Sep 17 00:00:00 2001 From: geeksville Date: Wed, 22 Apr 2020 08:20:57 -0700 Subject: [PATCH] move logging to the IO thread --- .../com/geeksville/mesh/service/RadioInterfaceService.kt | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/src/main/java/com/geeksville/mesh/service/RadioInterfaceService.kt b/app/src/main/java/com/geeksville/mesh/service/RadioInterfaceService.kt index 69936af06..ac95b6788 100644 --- a/app/src/main/java/com/geeksville/mesh/service/RadioInterfaceService.kt +++ b/app/src/main/java/com/geeksville/mesh/service/RadioInterfaceService.kt @@ -372,17 +372,18 @@ class RadioInterfaceService : Service(), Logging { // hasForcedRefresh = true forceServiceRefresh() } - + // we begin by setting our MTU size as high as it can go safe!!.asyncRequestMtu(512) { mtuRes -> mtuRes.getOrThrow() // FIXME - why sometimes is the result Unit!?! - debug("requested MTU result=$mtuRes") + debug("MTU change attempted") // FIXME - no need to discover services more than once - instead use lazy() to use them in future attempts safe!!.asyncDiscoverServices { discRes -> discRes.getOrThrow() // FIXME, instead just try to reconnect? - debug("Discovered services!") + serviceScope.handledLaunch { + debug("Discovered services!") delay(500) // android BLE is buggy and needs a 500ms sleep before calling getChracteristic, or you might get back null fromNum = service.getCharacteristic(BTM_FROMNUM_CHARACTER)!!