diff options
| author | Jeff King <peff@peff.net> | 2009-09-05 04:54:14 -0400 | 
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2009-09-05 23:16:27 -0700 | 
| commit | dd2be243d62260d4c825c22fdd2f61a7da12de22 (patch) | |
| tree | 6d709c59b43e2fe601ff21047b145d0dbcaf0226 /builtin-commit.c | |
| parent | 01d8ba187d6e8a6bfa908fbef16a36d1186981dd (diff) | |
| download | git-dd2be243d62260d4c825c22fdd2f61a7da12de22.tar.gz | |
status: refactor format option parsing
This makes it possible to have more than two formats.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-commit.c')
| -rw-r--r-- | builtin-commit.c | 21 | 
1 files changed, 14 insertions, 7 deletions
diff --git a/builtin-commit.c b/builtin-commit.c index 5b42179fe7..aa4a358799 100644 --- a/builtin-commit.c +++ b/builtin-commit.c @@ -991,12 +991,16 @@ static void short_print(struct wt_status *s, int null_termination)  int cmd_status(int argc, const char **argv, const char *prefix)  {  	struct wt_status s; -	static int null_termination, shortstatus; +	static int null_termination; +	static enum { +		STATUS_FORMAT_LONG, +		STATUS_FORMAT_SHORT, +	} status_format = STATUS_FORMAT_LONG;  	unsigned char sha1[20];  	static struct option builtin_status_options[] = {  		OPT__VERBOSE(&verbose), -		OPT_BOOLEAN('s', "short", &shortstatus, -			    "show status concisely"), +		OPT_SET_INT('s', "short", &status_format, +			    "show status concisely", STATUS_FORMAT_SHORT),  		OPT_BOOLEAN('z', "null", &null_termination,  			    "terminate entries with NUL"),  		{ OPTION_STRING, 'u', "untracked-files", &untracked_files_arg, @@ -1006,8 +1010,8 @@ int cmd_status(int argc, const char **argv, const char *prefix)  		OPT_END(),  	}; -	if (null_termination) -		shortstatus = 1; +	if (null_termination && status_format == STATUS_FORMAT_LONG) +		status_format = STATUS_FORMAT_SHORT;  	wt_status_prepare(&s);  	git_config(git_status_config, &s); @@ -1024,9 +1028,11 @@ int cmd_status(int argc, const char **argv, const char *prefix)  	s.is_initial = get_sha1(s.reference, sha1) ? 1 : 0;  	wt_status_collect(&s); -	if (shortstatus) +	switch (status_format) { +	case STATUS_FORMAT_SHORT:  		short_print(&s, null_termination); -	else { +		break; +	case STATUS_FORMAT_LONG:  		s.verbose = verbose;  		if (s.relative_paths)  			s.prefix = prefix; @@ -1035,6 +1041,7 @@ int cmd_status(int argc, const char **argv, const char *prefix)  		if (diff_use_color_default == -1)  			diff_use_color_default = git_use_color_default;  		wt_status_print(&s); +		break;  	}  	return 0;  }  | 
