Hi Inaky, On 07/23/2010 03:59 PM, Inaky Perez-Gonzalez wrote: > From: Inaky Perez-Gonzalez <inaky.perez-gonza...@intel.com> > > These have been stolen from the Linux kernel source; come pretty handy > to make build-time consistency checks and thus avoid run-time > surprises. > --- > src/bug.h | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++ > src/smsutil.c | 1 + > 2 files changed, 51 insertions(+), 0 deletions(-) > create mode 100644 src/bug.h >
First of all, please refer to doc/coding-style.txt Section M5. Second, please actually check that your code passes make distcheck: CCLD unit/test-idmap CC unit/test-sms.o CC src/smsutil.o ../src/smsutil.c:40:17: error: bug.h: No such file or directory make[2]: *** [src/smsutil.o] Error 1 make[1]: *** [all] Error 2 make: *** [distcheck] Error 1 > diff --git a/src/smsutil.c b/src/smsutil.c > index e41c041..6c2087a 100644 > --- a/src/smsutil.c > +++ b/src/smsutil.c > @@ -37,6 +37,7 @@ > #include "util.h" > #include "storage.h" > #include "smsutil.h" > +#include "bug.h" Why are you including this without actually using it? > > #define uninitialized_var(x) x = x > Regards, -Denis _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono