Resume the BT Connection from the original list if connection was not successfully established during resume so GC can free up the area

geza-pycom · geza-pycom · commit 9f4489611162 · 2020-11-03T17:54:41.000+01:00
diff --git a/esp32/mods/modbt.c b/esp32/mods/modbt.c
@@ -428,6 +428,10 @@ void bt_resume(bool reconnect)
                     // As modbt_connect appends the new connection to the original list, it needs to be removed because it is not needed
                     mp_obj_list_remove((void *)&MP_STATE_PORT(btc_conn_list), new_connection_obj);
                 }
+                else {
+                    // Remove the old connection from the original list because connection could not be established with it
+                    mp_obj_list_remove((void *)&MP_STATE_PORT(btc_conn_list), connection_obj);
+                }
             }
 
             /* See if there was an advertisement active before Sleep */

Original file line number	Diff line number	Diff line change
`@@ -428,6 +428,10 @@ void bt_resume(bool reconnect)`
`428`	`428`	`// As modbt_connect appends the new connection to the original list, it needs to be removed because it is not needed`
`429`	`429`	`mp_obj_list_remove((void *)&MP_STATE_PORT(btc_conn_list), new_connection_obj);`
`430`	`430`	`}`
	`431`	`+ else {`
	`432`	`+ // Remove the old connection from the original list because connection could not be established with it`
	`433`	`+ mp_obj_list_remove((void *)&MP_STATE_PORT(btc_conn_list), connection_obj);`
	`434`	`+ }`
`431`	`435`	`}`
`432`	`436`
`433`	`437`	`/* See if there was an advertisement active before Sleep */`