> -----Original Message----- > From: Brandon Williams [mailto:bmwill@xxxxxxxxxx] > > +struct scheduled_submodules_update_type { > > + const char *path; > > + const struct object_id *oid; > > + /* > > + * Do we need to perform a complete checkout or just incremental > > + * update? > > + */ > > + unsigned is_new:1; > > +} *scheduled_submodules; > > +#define SCHEDULED_SUBMODULES_INIT {NULL, NULL} > > I may not know enough about these types of initializors but that Init > macro only has 2 entries while there are three entries in the struct > itself. In fact, only the first NULL is necessary; unspecified initializer entries in C default to zero.