From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 07 Jun 2008 01:55:36 GMT
Subject: Re: [PATCH 1/3] Add an optional <mode> argument to commit/status -u|--untracked-files option
Message-ID: <7v4p86qa93.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <5f0ab026ce200e501be81a3b5082e482e1580e42.1212670149.git.marius@trolltech.com>

Marius Storm-Olsen <marius@trolltech.com> writes:

> diff --git a/builtin-commit.c b/builtin-commit.c
> index b294c1f..1f4986b 100644
> --- a/builtin-commit.c
> +++ b/builtin-commit.c
> @@ -102,7 +103,7 @@ static struct option builtin_commit_options[] = {
>  	OPT_BOOLEAN('o', "only", &only, "commit only specified files"),
>  	OPT_BOOLEAN('n', "no-verify", &no_verify, "bypass pre-commit hook"),
>  	OPT_BOOLEAN(0, "amend", &amend, "amend previous commit"),
> -	OPT_BOOLEAN('u', "untracked-files", &untracked_files, "show all untracked files"),
> +	{ OPTION_STRING, 'u', "untracked-files", &untracked_files_arg, "mode", "show untracked files, optional modes: all, normal. (Default: all)", PARSE_OPT_OPTARG, NULL, (int)"all" },

Hmm.

$ make
builtin-commit.c:106: error: initializer element is not constant
builtin-commit.c:106: error: (near initialization for 'builtin_commit_options[18].defval')
make: *** [builtin-commit.o] Error 1

I also have to wonder what the funny cast of (int)"all"  is doing.