Hi Johan, On Tue, Mar 15, 2011 at 2:54 PM, <johan.hedberg@xxxxxxxxx> wrote: > From: Johan Hedberg <johan.hedberg@xxxxxxxxx> > > This patch adds a new set_local_name management command as well as a > local_name_changed management event. With these user space can both > change the local name as well as monitor changes to it by others. > > The management messages reserve 249 bytes for the name instead of 248 > (like in the HCI spec) so that there is always a guarantee that it is > nul-terminated. That way it can safely be passed onto string > manipulation functions. I wonder if it is "future-proof" to use 248 and 249 along the code without using some #define for the the spec defined value. Someone may look at the code later and think there is a bug, even though it is clear from the commit message. My two cents, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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
