fix autobug: some phones call gatt connected but with an error status code

app/src/main/java/com/geeksville/mesh/service/RadioInterfaceService.kt

@@ -354,7 +354,7 @@ class RadioInterfaceService : Service(), Logging {
     private fun onConnect(connRes: Result<Unit>) {
         // This callback is invoked after we are connected
 
-        connRes.getOrThrow() // FIXME, instead just try to reconnect?
+        connRes.getOrThrow()
         info("Connected to radio!")
 
         if (!hasForcedRefresh) {

app/src/main/java/com/geeksville/mesh/service/SafeBluetooth.kt

@@ -154,9 +154,11 @@ class SafeBluetooth(private val context: Context, private val device: BluetoothD
                 BluetoothProfile.STATE_CONNECTED -> {
                     state =
                         newState // we only care about connected/disconnected - not the transitional states
-                    //logAssert(workQueue.isNotEmpty())
-                    //val work = workQueue.removeAt(0)
-                    completeWork(status, Unit)
+
+                    if (status != BluetoothGatt.GATT_SUCCESS) {
+                        errormsg("Connect attempt failed $status, not calling connect completion handler...")
+                    } else
+                        completeWork(status, Unit)
                 }
                 BluetoothProfile.STATE_DISCONNECTED -> {
                     // cancel any queued ops if we were already connected

geeksville-androidlib

@@ -1 +1 @@
-Subproject commit bfe039fa19c0625b3f81f30a614c594e9c6979aa
+Subproject commit b052e3bf0708dbc075b9017c19af6833dc6fa4f9