diff options
| author | Mikel Astiz <mikel.astiz@bmw-carit.de> | 2012-07-26 12:36:33 +0200 |
|---|---|---|
| committer | Jaska Uimonen <jaska.uimonen@intel.com> | 2013-01-24 09:29:35 +0200 |
| commit | 7b3310fb9bfc5d4d4f3efa9e8d48818a7c599237 (patch) | |
| tree | d19de6a98a2e2f48bb84a78dff0f7baa65363130 /src | |
| parent | 68728df301624e193b8d095f390de7847f2bf99e (diff) | |
| download | pulseaudio-panda-7b3310fb9bfc5d4d4f3efa9e8d48818a7c599237.tar.gz pulseaudio-panda-7b3310fb9bfc5d4d4f3efa9e8d48818a7c599237.tar.bz2 pulseaudio-panda-7b3310fb9bfc5d4d4f3efa9e8d48818a7c599237.zip | |
bluetooth: Remove minor unnecessary check
The return value of dbus_message_iter_next() doesn't need to be checked
since the while condition will be false anyway (arg type will be
DBUS_TYPE_INVALID).
Diffstat (limited to 'src')
| -rw-r--r-- | src/modules/bluetooth/bluetooth-util.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/modules/bluetooth/bluetooth-util.c b/src/modules/bluetooth/bluetooth-util.c index bb32ad25..bfe00dcf 100644 --- a/src/modules/bluetooth/bluetooth-util.c +++ b/src/modules/bluetooth/bluetooth-util.c @@ -396,8 +396,7 @@ static int parse_device_property(pa_bluetooth_discovery *y, pa_bluetooth_device has_audio = TRUE; } - if (!dbus_message_iter_next(&ai)) - break; + dbus_message_iter_next(&ai); } /* this might eventually be racy if .Audio is not there yet, but the State change will come anyway later, so this call is for cold-detection mostly */ @@ -589,8 +588,7 @@ static void get_properties_reply(DBusPendingCall *pending, void *userdata) { } } - if (!dbus_message_iter_next(&element_i)) - break; + dbus_message_iter_next(&element_i); } finish: |