From: Brandon Casey Date: Mon, 05 Nov 2007 23:08:30 GMT Subject: Re: [PATCH] gc: use parse_options Message-ID: <472FA26E.4060706@nrlssc.navy.mil> In-Reply-To: <7vhck579pm.fsf@gitster.siamese.dyndns.org> Junio C Hamano wrote: > James Bowes 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" -brandon