From: Mikel Astiz <mikel.astiz@xxxxxxxxxxxx> The Adapter interface already reports changes in the device list in form of property changes, so there is no need to keep these two signals. --- doc/adapter-api.txt | 8 -------- src/adapter.c | 17 ----------------- 2 files changed, 25 deletions(-) diff --git a/doc/adapter-api.txt b/doc/adapter-api.txt index b638586..132e60f 100644 --- a/doc/adapter-api.txt +++ b/doc/adapter-api.txt @@ -116,14 +116,6 @@ Signals DevicesFound(array{object path, dict values}) The dictionary contains the properties from the org.bluez.Device interface. - DeviceCreated(object device) - - Parameter is object path of created device. - - DeviceRemoved(object device) - - Parameter is object path of removed device. - Properties string Address [readonly] The Bluetooth device address. diff --git a/src/adapter.c b/src/adapter.c index f4bb622..1f02f0b 100644 --- a/src/adapter.c +++ b/src/adapter.c @@ -1014,7 +1014,6 @@ static struct btd_device *adapter_create_device(struct btd_adapter *adapter, uint8_t bdaddr_type) { struct btd_device *device; - const char *path; DBG("%s", address); @@ -1026,12 +1025,6 @@ static struct btd_device *adapter_create_device(struct btd_adapter *adapter, adapter->devices = g_slist_append(adapter->devices, device); - path = device_get_path(device); - g_dbus_emit_signal(btd_get_dbus_connection(), adapter->path, - ADAPTER_INTERFACE, "DeviceCreated", - DBUS_TYPE_OBJECT_PATH, &path, - DBUS_TYPE_INVALID); - g_dbus_emit_property_changed(btd_get_dbus_connection(), adapter->path, ADAPTER_INTERFACE, "Devices"); @@ -1059,7 +1052,6 @@ void adapter_remove_device(struct btd_adapter *adapter, struct btd_device *dev, gboolean remove_storage) { - const gchar *dev_path = device_get_path(dev); struct discovery *discovery = adapter->discovery; GList *l; @@ -1091,11 +1083,6 @@ void adapter_remove_device(struct btd_adapter *adapter, g_dbus_emit_property_changed(btd_get_dbus_connection(), adapter->path, ADAPTER_INTERFACE, "Devices"); - g_dbus_emit_signal(btd_get_dbus_connection(), adapter->path, - ADAPTER_INTERFACE, "DeviceRemoved", - DBUS_TYPE_OBJECT_PATH, &dev_path, - DBUS_TYPE_INVALID); - device_remove(dev, remove_storage); } @@ -1687,10 +1674,6 @@ static const GDBusMethodTable adapter_methods[] = { }; static const GDBusSignalTable adapter_signals[] = { - { GDBUS_SIGNAL("DeviceCreated", - GDBUS_ARGS({ "device", "o" })) }, - { GDBUS_SIGNAL("DeviceRemoved", - GDBUS_ARGS({ "device", "o" })) }, { GDBUS_SIGNAL("DevicesFound", GDBUS_ARGS({ "devices", "a{oa{sv}}" })) }, { } -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html