From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 06 Nov 2007 00:37:08 GMT
Subject: Re: [PATCH] gc: use parse_options
Message-ID: <7vr6j4ky4r.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <472FA26E.4060706@nrlssc.navy.mil>

Brandon Casey <casey@nrlssc.navy.mil> writes:

> Junio C Hamano wrote:
>> James Bowes <jbowes@dangerouslyinc.com> writes:
>> 
>>> +	struct option builtin_gc_options[] = {
>>> +		OPT_BOOLEAN(0, "prune", &prune, "prune unused objects"),
>> 
>> I would write "unreferenced loose" instead of "unused" here...
>
> It is not just "loose" objects here, but also unreferenced objects in packs,
> since the "-a" option to repack is now only used when --prune is specified.
> Without --prune, "-A" is supplied to repack instead.
>
> So maybe the message should just be "prune unreferenced objects"

Fair enough, will do.