[pulseaudio-discuss] [PATCH v2 1/9] bluetooth: Use assertions when setting off profile

Mikel Astiz mikel.astiz.oss at gmail.com
Fri Oct 19 01:11:21 PDT 2012


From: Mikel Astiz <mikel.astiz at bmw-carit.de>

Setting the card profile to off cannot fail, therefore assertions can be
used when calling pa_card_set_profile().
---
 src/modules/bluetooth/module-bluetooth-device.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/modules/bluetooth/module-bluetooth-device.c b/src/modules/bluetooth/module-bluetooth-device.c
index d78ce8c..ee17fde 100644
--- a/src/modules/bluetooth/module-bluetooth-device.c
+++ b/src/modules/bluetooth/module-bluetooth-device.c
@@ -605,8 +605,7 @@ static int device_process_msg(pa_msgobject *obj, int code, void *data, int64_t o
 
             pa_log_debug("Switching the profile to off due to IO thread failure.");
 
-            if (pa_card_set_profile(u->card, "off", FALSE) < 0)
-                pa_log_debug("Failed to switch profile to off");
+            pa_assert_se(pa_card_set_profile(u->card, "off", false) >= 0);
             break;
         }
     }
-- 
1.7.11.7



More information about the pulseaudio-discuss mailing list