Hi Johan,
2009/8/16 Johan Hedberg <johan.hedberg@xxxxxxxxx>:
> Hi Luiz,
>
> On Sun, Aug 16, 2009, Luiz Augusto von Dentz wrote:
>> > On Sat, Aug 15, 2009, João Paulo Rechi Vita wrote:
>> >> if (!interface) {
>> >> - if (dev->sink && avdtp_is_connected(&dev->src, &dev->dst))
>> >> + if (dev->source && avdtp_is_connected(&dev->src, &dev->dst))
>> >> + return TYPE_SOURCE;
>> >> + else if (dev->sink && avdtp_is_connected(&dev->src, &dev->dst))
>> >> return TYPE_SINK;
>> >
>> > I think the dev->source check should be in an "else if" clause while the
>> > dev->sink check should be first. I.e. if a device happens to support both
>> > roles of A2DP we default to dev->sink.
>>
>> Good catches, perhaps we should consider doing avdtp_is_connected
>> check before checking for sink/source pointers. What do you think?
>
> What exactly do you mean? Could you perhaps give an example code snippet
> of how you'd like the code to look like?
>
> Johan
>
Something like this:
if (avdtp_is_connected(...)) {
if (dev->sink) return TYPE_SINK;
else if (dev->source) return TYPE_SOURCE;
else...
}
--
Luiz Augusto von Dentz
Engenheiro de Computação
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
