On Tue, Feb 02, 2010 at 09:56:37AM +0200, Tim Just wrote: > > --- modules/update/hipd/update_init.h 1970-01-01 00:00:00 +0000 > +++ modules/update/hipd/update_init.h 2010-02-02 07:55:10 +0000 > @@ -0,0 +1,13 @@ > + > +#ifndef UPDATE_INIT_H_ > +#define UPDATE_INIT_H_ The _ postfix on the multiple inclusion guard just looks weird IMO. Also, multiple inclusion guards should carry a prefix to avoid name clashes, i.e. HIPL_. Diego