Hi Brian, On 14 Mar 2011, at 17:57 , Brian Gix wrote: > Hi Elvis, > > On 3/14/2011 11:53 AM, Elvis Pfützenreuter wrote: >> From: Claudio Takahasi<claudio.takahasi@xxxxxxxxxxxxx> >> >> 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. Thanks for the review, I will send a fixed version of this patch only.-- 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