On Sat, Oct 13, 2007 at 02:47:20PM +0000, Johannes Schindelin wrote:
quoted text
> Hi,
>=20
> On Sat, 13 Oct 2007, Pierre Habouzit wrote:
>=20
> > +static struct option builtin_add_options[] =3D {
> > +	OPT_BOOLEAN('i', "interactive", &add_interactive, "interactive pickin=
g"),
quoted text
> > +	OPT_BOOLEAN('n', NULL, &show_only, "dry-run"),
> > +	OPT_BOOLEAN('f', NULL, &ignored_too, "allow adding otherwise ignored =
files"),
quoted text
> > +	OPT_BOOLEAN('v', NULL, &verbose, "be verbose"),
> > +	OPT_BOOLEAN('u', NULL, &take_worktree_changes, "update only files tha=
t git already knows about"),
quoted text
> > +	OPT_BOOLEAN( 0 , "refresh", &refresh_only, "don't add, only refresh s=
tat() informations in the index"),
quoted text
> > +};
>=20
> I see you terminate the list by a ",".  How does this play with the optio=
n=20
quoted text
> parser?
>=20
> Thinking about this more, I am reverting my stance on the ARRAY_SIZE()=20
> issue.  I think if you introduce a "OPTION_NONE =3D 0" in the enum, then=
=20
quoted text
> this single last comma should be enough.

  adding a trailing comma does not add a NULL after that, it's ignored,
you're confused.

    =E2=94=8C=E2=94=80(17:00)=E2=94=80=E2=94=80=E2=94=80=E2=94=80
    =E2=94=94[artemis] cat a.c
    #include <stdio.h>

      int main(void) {
	const char * const arr[] =3D { "1", "2", };
	printf("%d\n", sizeof(arr) / sizeof(arr[0]));
	return 0;
    };
    =E2=94=8C=E2=94=80(17:00)=E2=94=80=E2=94=80=E2=94=80=E2=94=80
    =E2=94=94[artemis] ./a
    2

  Very few compilers do not grok trailing commas, I always put them
because it avoids spurious diffs for nothing, and that you can reorder
lines easily too.

  Note that I don't really like using ARRAY_SIZE either, I kept it that
way, but my taste would rather be to have an "empty" option, and
explicitely mark the end of the array.

--=20
=C2=B7O=C2=B7  Pierre Habouzit
=C2=B7=C2=B7O                                                madcoder@debia=
n.org
OOO                                                http://www.madism.org