From: Jeff King <peff@peff.net>
Date: Wed, 30 Mar 2011 03:32:05 GMT
Subject: Re: [PATCH 7/6] log/pretty-options: Document --[no-]notes and deprecate old notes options
Message-ID: <20110330033205.GB18157@sigill.intra.peff.net>
In-Reply-To: <201103300257.19921.johan@herland.net>

On Wed, Mar 30, 2011 at 02:57:19AM +0200, Johan Herland wrote:

> On Wednesday 30 March 2011, Jeff King wrote:
> > On Wed, Mar 30, 2011 at 12:18:55AM +0200, Johan Herland wrote:
> > > And here's some documentation to go on top.
> > [...]
> > This example is wrong. "--notes=foo" will show _only_ foo. A few
> > examples:
> > 
> >   --notes=foo ;# only foo
> >   --notes --notes=foo ;# foo and default
> >   --notes=foo --notes=bar --notes --no-notes --notes=foo ;# only foo
> > 
> > Make sense?
> 
> Yeah, I really screwed that up. Hope this version is better. :)

Yes, it looks good to me. Thanks.

Acked-by: Jeff King <peff@peff.net>

-Peff