Hi Anderson, On Thu, May 9, 2013 at 7:55 AM, Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx> wrote: > Hi Lucas, > > On Thu, May 9, 2013 at 12:16 AM, Lucas De Marchi > <lucas.demarchi@xxxxxxxxxxxxxx> wrote: >> diff --git a/attrib/gatt.h b/attrib/gatt.h >> index 305b4c6..d5758b7 100644 >> --- a/attrib/gatt.h >> +++ b/attrib/gatt.h >> @@ -22,6 +22,7 @@ >> * >> */ >> >> +#include <inttypes.h> >> #include <bluetooth/sdp.h> > > We used to avoid adding includes into headers, and instead add them on > the .c files only. (The existing include above is a unfortunate > exception). Not sure if this convention has changed since. Sure, but when you use a certain type in header you need the header (or if it's a pointer you have the option to do a forward declaration, but it's not the case here). If you don't do this you depend on the include order in each .c, risking to break the build, which is way worse. 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
