Hi, On Thu, Nov 11, 2010, Vinicius Costa Gomes wrote: > + int8_t rssi = 0; > + > + rssi = *(info->data + info->length); There's no point in initializing rssi upon declaration since the very next thing you do is assign a new value to it (not to mention that in general we try to avoid initialization upon declaration in the code). > +void btd_event_adv(bdaddr_t *local, le_advertising_info *info) I'm not really a fan of the adv abreviation, and the function name isn't even in danger of growing horribly long if you don't abreviate it. So could you come up with something more clear? Maybe btd_event_advertising_info or btd_event_advertising_report? I'll stop reviewing this patch-set here since the rest seem to depend on the name of this function. 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
