OSDN Git Service

avctp: Remove double looking for audio device
authorSzymon Janc <szymon.janc@tieto.com>
Wed, 12 Dec 2012 10:28:08 +0000 (11:28 +0100)
committerLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Wed, 19 Dec 2012 11:57:25 +0000 (13:57 +0200)
There is no need to double call for manager_get_audio_device with
toggled create flag. manager_get_audio_device with create set to true
will only create new audio device if matching one was not found.

profiles/audio/avctp.c

index 8ffbf43..013c587 100644 (file)
@@ -1132,14 +1132,10 @@ static void avctp_confirm_cb(GIOChannel *chan, gpointer data)
        if (session == NULL)
                return;
 
-       dev = manager_get_audio_device(device, FALSE);
+       dev = manager_get_audio_device(device, TRUE);
        if (!dev) {
-               dev = manager_get_audio_device(device, TRUE);
-               if (!dev) {
-                       error("Unable to get audio device object for %s",
-                                       address);
-                       goto drop;
-               }
+               error("Unable to get audio device object for %s", address);
+               goto drop;
        }
 
        if (dev->control == NULL) {