Re: [PATCH] Emit Connect signal for LE capable devices

[Sheldon Demario <sheldon.demario@openbossa.org>]

On 11/29/2010 07:52 AM, Johan Hedberg wrote:
> Hi Sheldon,
>
> On Thu, Nov 25, 2010, Sheldon Demario wrote:
>> +	if (le) {
>> +		evt_le_connection_complete *evt = ptr;
>> +		evt_bdaddr =&evt->peer_bdaddr;
>> +		evt_handle = evt->handle;
>> +		evt_status = evt->status;
>> +	}  else {
>> +		evt_conn_complete *evt = ptr;
>> +		evt_bdaddr =&evt->bdaddr;
>> +		evt_handle = evt->handle;
>> +		evt_status = evt->status;
>> +
>> +		if (evt->link_type != ACL_LINK)
>> +			return;
>> +	}
> Instead of this kind of trickery, I have a feeling that the code would
> be easier to read if you had a separate function for the LE connect
> complete. Could try try to come up with a patch that does it like that?

For sure, but don't you think that doing this way there will be a lot of 
duplicated code?

Sheldon.