Re: [PATCH v2] Documentation: avoid dangling modifier for imap-send

From: brian m. carlson <sandals@crustytoothpaste.net>
Date: Aug 5, 2014, 00:54 UTC
Message-ID: <20140805005451.GA10435@vauxhall.crustytoothpaste.net>
In-Reply-To: <xmqq8un4rtv6.fsf@gitster.dls.corp.google.com>

On Mon, Aug 04, 2014 at 11:29:33AM -0700, Junio C Hamano wrote:

Show 24 quoted lines

> "brian m. carlson" <sandals@crustytoothpaste.net> writes:
> 
> > Avoid a nonsensical misreading by moving the modifier closer to the
> > word it modifies.
> >
> > Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
> > ---
> >  Documentation/git-imap-send.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/git-imap-send.txt b/Documentation/git-imap-send.txt
> > index 875d283..23231e1 100644
> > --- a/Documentation/git-imap-send.txt
> > +++ b/Documentation/git-imap-send.txt
> > @@ -43,7 +43,7 @@ imap.folder::
> >  imap.tunnel::
> >  	Command used to setup a tunnel to the IMAP server through which
> >  	commands will be piped instead of using a direct network connection
> > -	to the server. Required when imap.host is not set to use imap-send.
> > +	to the server. Required to use imap-send when imap.host is not set.
> 
> To be honest, I find both versions are equally confusing.
> 
> How about dropping the three words "to use imap-send"?

Fine by me.

-- 
brian m. carlson / brian with sandals: Houston, Texas, US
+1 832 623 2791 | http://www.crustytoothpaste.net/~bmc | My opinion only
OpenPGP: RSA v4 4096b: 88AC E9B2 9196 305B A994 7552 F1BA 225C 0223 B187