Re: [PATCH BlueZ v2 04/14] core: Add a list of LE devices to connect

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi João Paulo,

On Wed, Aug 15, 2012, João Paulo Rechi Vita wrote:
> +	device_get_address(device, &bdaddr, NULL);
> +	if (g_slist_find_custom(adapter->connect_list, &bdaddr,
> +					(GCompareFunc) device_bdaddr_cmp)) {
> +		DBG("trying to add device %s which is already in the connect "
> +				"list, ignoring", device_get_path(device));
> +		return;
> +	}

Why don't you just use g_slist_find here? If this is the only place
where you need the new device_bdaddr_cmp then patch 03/14 can be
removed.

Johan
--
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux