From: Junio C Hamano Date: Wed, 27 Aug 2008 17:19:09 GMT Subject: Re: [PATCH v2] allow user aliases for the --author parameter Message-ID: <7viqtmqstu.fsf@gitster.siamese.dyndns.org> In-Reply-To: <20080827122954.GA11986@coredump.intra.peff.net> Jeff King writes: > My only complaint is: > >> + strbuf_addf(&buf, "--author=%s", name); >> + av[++ac] = "--all"; >> + av[++ac] = buf.buf; >> + av[++ac] = NULL; > > I am too lazy to hit "shift", so I would use "-i". I thought about it after writing the one you saw on the list, but from the beginning I was planning to do this merely as a demonstration patch that somebody who is interested in the feature can polish and resubmit with test and documentation. I didn't bother adding such frills --- that is part of "polish and resubmit" cycle.