On 08/22/2012 10:08 AM, Mikel Astiz wrote:
From: Mikel Astiz <[email protected]>
A port with the availability state flag set to unknown should not be
considered an active port.
---
src/modules/module-switch-on-port-available.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/modules/module-switch-on-port-available.c
b/src/modules/module-switch-on-port-available.c
index 1da612d..8c16821 100644
--- a/src/modules/module-switch-on-port-available.c
+++ b/src/modules/module-switch-on-port-available.c
@@ -85,7 +85,7 @@ static pa_bool_t try_to_switch_profile(pa_card *card,
pa_device_port *port) {
PA_IDXSET_FOREACH(sink, card->sinks, state2) {
if (!sink->active_port)
continue;
- if (sink->active_port->available != PA_PORT_AVAILABLE_NO)
+ if (sink->active_port->available == PA_PORT_AVAILABLE_YES)
found_active_port = TRUE;
}
I think this will break the following use case (on a normal laptop) :
Assume we plug in HDMI. With this patch we will get inconsistent
behaviour: if headphones are plugged in (available = yes), we won't
switch to HDMI, but if not (speakers are available = unknown), we will
switch to HDMI.
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
