From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 23 Jul 2007 07:41:13 GMT
Subject: Re: [PATCH] describe: Add unlisted option
Message-ID: <7vabtnmusm.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20070723070308.GH32566@spearce.org>

"Shawn O. Pearce" <spearce@spearce.org> writes:

> Yasushi SHOJI <yashi@atmark-techno.com> wrote:
>> added unlisted options, --contains, --candidates and --debug, to usage
>> line.  also, prints some info when --contains and --debug are given.
>
> That makes sense.  Especially telling the user why --debug --contains
> doesn't actually print anything.  ;-)
>
> Originally I left out --candidates and --debug when I wrote the code
> for them as I thought they were a tad too internal for casual use.
> But maybe it makes sense to include them in the usage string.

I was actually going to suggest removing these options, that
were primarily meant for debugging and tweaking while we figure
out what the optimum default should be.  Do you think they are
worth keeping?