From: Jonathan Nieder <jrnieder@gmail.com>
Date: Fri, 06 Aug 2010 21:04:29 GMT
Subject: Re: [PATCH] prune: allow --dry-run for -n and --verbose for -v
Message-ID: <20100806210429.GA6495@burratino>
In-Reply-To: <4C5C7055.3090100@lsrfire.ath.cx>

René Scharfe wrote:

> +++ b/builtin/prune.c
> @@ -125,10 +125,9 @@ int cmd_prune(int argc, const char **argv, const char *prefix)
>  {
>  	struct rev_info revs;
>  	const struct option options[] = {
> -		OPT_BOOLEAN('n', NULL, &show_only,
> +		OPT_BOOLEAN('n', "dry-run", &show_only,

For the curious: avoiding OPT__DRY_RUN allows us to give a better
description:

>  			    "do not remove, show only"),

which makes sense to me.  Thanks.