Hi Marcel, On Fri, Jul 1, 2011 at 6:59 PM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: > > Hi Lucas, > > > This provides a simple ARRAY_SIZE() macro, which (given a good compiler) > > will also break compile if you try to use it on a pointer. This can > > ensure your code is robust to changes, without needing a gratuitous > > macro or constant. Credits to Rusty Russell > > (http://ccan.ozlabs.org/info/array_size.html) > > > > Having a util.h file, this kind of things can be shared among all users > > and we can remove all the declarations of ARRAY_SIZE. > > --- > > acinclude.m4 | 22 ++++++++++++++++++++++ > > configure.ac | 1 + > > src/util.h | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 78 insertions(+), 0 deletions(-) > > create mode 100644 src/util.h > > I am not sure that I actually want this. Where are the use cases of > ARRAY_SIZE actually. Maybe we should just change them with proper NULL > terminated arrays or other constants. The uses cases are in the second patch. I added this because I was going to use it for the AVRCP patches. Why do you prefer NULL terminated arrays in case the content is static? Regards, Lucas De Marchi -- 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
