Re: [PATCH v3 1/5] Add new UUID utility functions

[Brian Gix <bgix@codeaurora.org>]

Hi Elvis,

On 3/14/2011 11:53 AM, Elvis Pfützenreuter wrote:
> From: Claudio Takahasi<claudio.takahasi@openbossa.org>
>
> New UUID functions will store the UUIDs values on host order. Added
> functions to create, compare and convert UUIDs.

[...]

> +
> +void bt_uuid_to_uuid128(const bt_uuid_t *src, bt_uuid_t *dst)
> +{
> +	switch (src->type) {
> +	case BT_UUID128:
> +		memcpy(dst, src, sizeof(bt_uuid_t));
> +		dst->type = BT_UUID128;

Line not needed. It will get copied with the rest of the structure 
during memcpy.

> +		break;
> +	case BT_UUID32:
> +		bt_uuid32_to_uuid128(src, dst);
> +		break;
> +	case BT_UUID16:
> +		bt_uuid16_to_uuid128(src, dst);
> +		break;
> +	default:
> +		break;
> +	}
> +}
> +

Regards,


-- 
Brian Gix
bgix@codeaurora.org
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum