Hi Hannes,
On Tue, 30 Aug 2016, Johannes Sixt wrote:
> Am 29.08.2016 um 23:59 schrieb Jakub Narębski:
> > W dniu 29.08.2016 o 10:04, Johannes Schindelin pisze:
> > > -#define REPLAY_OPTS_INIT { -1, -1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, NULL,
> > > NULL, NULL, 0, 0, NULL }
> > > +#define REPLAY_OPTS_INIT { -1, -1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, NULL,
> > > NULL, NULL, 0, 0, NULL, NULL, 0, 0 }
> >
> > Nb. it is a pity that we cannot use named initializers for structs,
> > so called designated inits. It would make this macro more readable.
>
> It is actually pointless to add the 0's and NULL's here. This should be
> sufficient:
>
> #define REPLAY_OPTS_INIT { -1, -1 }
>
> because initialization with 0 (or NULL) is the default for any omitted
> members.
D'oh. You're right. The same applies to TODO_LIST_INIT, of course.
Fixed,
Johannes
