[pulseaudio-discuss] [PATCH 04/56] Revert "bluetooth: Support media transport's State property"
jprvita at gmail.com
jprvita at gmail.com
Fri Jul 12 11:06:19 PDT 2013
From: João Paulo Rechi Vita <jprvita at openbossa.org>
This reverts commit 6fdf2b05b8867d4597c497243b469ecd0d8c2426.
This is part of the reversion of BlueZ 5 support so it can be added back
in a separate set of modules. This makes the code easier to maintain and
decrease PulseAudio's binary size.
---
src/modules/bluetooth/bluetooth-util.c | 42 +---------------------------------
1 file changed, 1 insertion(+), 41 deletions(-)
diff --git a/src/modules/bluetooth/bluetooth-util.c b/src/modules/bluetooth/bluetooth-util.c
index 6a21bc2..79e02c1 100644
--- a/src/modules/bluetooth/bluetooth-util.c
+++ b/src/modules/bluetooth/bluetooth-util.c
@@ -126,20 +126,6 @@ static pa_bt_audio_state_t audio_state_from_string(const char* value) {
return PA_BT_AUDIO_STATE_INVALID;
}
-static int transport_state_from_string(const char* value, pa_bluetooth_transport_state_t *state) {
- pa_assert(value);
- pa_assert(state);
-
- if (pa_streq(value, "idle"))
- *state = PA_BLUETOOTH_TRANSPORT_STATE_IDLE;
- else if (pa_streq(value, "pending") || pa_streq(value, "active")) /* We don't need such a separation */
- *state = PA_BLUETOOTH_TRANSPORT_STATE_PLAYING;
- else
- return -1;
-
- return 0;
-}
-
const char *pa_bt_profile_to_string(enum profile profile) {
switch(profile) {
case PROFILE_A2DP:
@@ -1099,29 +1085,6 @@ static int transport_parse_property(pa_bluetooth_transport *t, DBusMessageIter *
break;
}
-
- case DBUS_TYPE_STRING: {
-
- const char *value;
- dbus_message_iter_get_basic(&variant_i, &value);
-
- if (pa_streq(key, "State")) { /* Added in BlueZ 5.0 */
- bool old_any_connected = pa_bluetooth_device_any_audio_connected(t->device);
-
- if (transport_state_from_string(value, &t->state) < 0) {
- pa_log("Transport %s has an invalid state: '%s'", t->path, value);
- return -1;
- }
-
- pa_log_debug("dbus: transport %s set to state '%s'", t->path, value);
- pa_hook_fire(&t->device->discovery->hooks[PA_BLUETOOTH_HOOK_TRANSPORT_STATE_CHANGED], t);
-
- if (old_any_connected != pa_bluetooth_device_any_audio_connected(t->device))
- run_callback(t->device, old_any_connected);
- }
-
- break;
- }
}
return 0;
@@ -1600,10 +1563,7 @@ static pa_bluetooth_transport *transport_new(pa_bluetooth_device *d, const char
memcpy(t->config, config, size);
}
- if (d->discovery->version == BLUEZ_VERSION_4)
- t->state = audio_state_to_transport_state(d->profile_state[p]);
- else
- t->state = PA_BLUETOOTH_TRANSPORT_STATE_IDLE;
+ t->state = audio_state_to_transport_state(d->profile_state[p]);
return t;
}
--
1.7.11.7
More information about the pulseaudio-discuss
mailing list