[gnome-bluetooth/wip/hadess/workaround-connect-firmware: 5/5] settings-widget: Work-around intermittent connection problems
- From: Bastien Nocera <hadess src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-bluetooth/wip/hadess/workaround-connect-firmware: 5/5] settings-widget: Work-around intermittent connection problems
- Date: Sun, 20 Sep 2020 22:46:43 +0000 (UTC)
commit 0e79159d866d10292791530997e05a1a3cad7092
Author: Bastien Nocera <hadess hadess net>
Date: Wed Sep 16 14:07:04 2020 +0200
settings-widget: Work-around intermittent connection problems
Disable discovery during pairing and connect so as to work around broken
firmwares and hardware not being able to correctly connect to remote
devices when the adapter is performing a discovery.
Closes: https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/820
lib/bluetooth-settings-widget.c | 31 ++++++++++++++++++++++++++++++-
1 file changed, 30 insertions(+), 1 deletion(-)
---
diff --git a/lib/bluetooth-settings-widget.c b/lib/bluetooth-settings-widget.c
index 8577b0f4..5e9d3ca2 100644
--- a/lib/bluetooth-settings-widget.c
+++ b/lib/bluetooth-settings-widget.c
@@ -162,6 +162,17 @@ is_connecting (BluetoothSettingsWidget *self,
bdaddr));
}
+static gboolean
+has_default_adapter (BluetoothSettingsWidget *self)
+{
+ g_autofree char *default_adapter = NULL;
+ BluetoothSettingsWidgetPrivate *priv;
+
+ priv = BLUETOOTH_SETTINGS_WIDGET_GET_PRIVATE (self);
+ g_object_get (priv->client, "default-adapter", &default_adapter, NULL);
+ return (default_adapter != NULL);
+}
+
typedef struct {
char *bdaddr;
BluetoothSettingsWidget *self;
@@ -205,6 +216,9 @@ connect_done (GObject *source_object,
remove_connecting (self, data->bdaddr);
//FIXME show an error if it failed?
+ g_object_set (G_OBJECT (priv->client),
+ "default-adapter-discovering", has_default_adapter (self),
+ NULL);
out:
g_clear_error (&error);
@@ -856,6 +870,7 @@ connect_callback (GObject *source_object,
gpointer user_data)
{
SetupConnectData *data = (SetupConnectData *) user_data;
+ BluetoothSettingsWidgetPrivate *priv;
GError *error = NULL;
gboolean success;
@@ -876,6 +891,11 @@ connect_callback (GObject *source_object,
turn_off_pairing (data->self, data->device);
+ priv = BLUETOOTH_SETTINGS_WIDGET_GET_PRIVATE (data->self);
+ g_object_set (G_OBJECT (priv->client),
+ "default-adapter-discovering", has_default_adapter (data->self),
+ NULL);
+
bail:
if (data->timeout_id > 0)
g_source_remove (data->timeout_id);
@@ -890,7 +910,8 @@ create_callback (GObject *source_object,
GAsyncResult *res,
gpointer user_data)
{
- BluetoothSettingsWidgetPrivate *priv = BLUETOOTH_SETTINGS_WIDGET_GET_PRIVATE (user_data);
+ BluetoothSettingsWidget *self = user_data;
+ BluetoothSettingsWidgetPrivate *priv = BLUETOOTH_SETTINGS_WIDGET_GET_PRIVATE (self);
SetupConnectData *data;
GError *error = NULL;
gboolean ret;
@@ -938,6 +959,10 @@ create_callback (GObject *source_object,
g_free (dbus_error);
g_error_free (error);
g_free (path);
+
+ g_object_set (G_OBJECT (priv->client),
+ "default-adapter-discovering", has_default_adapter (self),
+ NULL);
return;
}
@@ -1033,6 +1058,7 @@ start_pairing (BluetoothSettingsWidget *self,
g_strdup (g_dbus_proxy_get_object_path (proxy)),
GINT_TO_POINTER (1));
+ g_object_set (G_OBJECT (priv->client), "default-adapter-discovering", FALSE, NULL);
bluetooth_client_setup_device (priv->client,
g_dbus_proxy_get_object_path (proxy),
pair,
@@ -1060,6 +1086,9 @@ switch_connected_state_set (GtkSwitch *button,
data->bdaddr = g_strdup (priv->selected_bdaddr);
data->self = self;
+ g_object_set (G_OBJECT (priv->client),
+ "default-adapter-discovering", has_default_adapter (self),
+ NULL);
bluetooth_client_connect_service (priv->client,
priv->selected_object_path,
gtk_switch_get_active (button),
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]