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

2014-08-04 Thread Junio C Hamano
brian m. carlson sand...@crustytoothpaste.net writes: Avoid a nonsensical misreading by moving the modifier closer to the word it modifies. Signed-off-by: brian m. carlson sand...@crustytoothpaste.net --- Documentation/git-imap-send.txt | 2 +- 1 file changed, 1 insertion(+), 1

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

2014-08-04 Thread brian m. carlson
On Mon, Aug 04, 2014 at 11:29:33AM -0700, Junio C Hamano wrote: brian m. carlson sand...@crustytoothpaste.net writes: Avoid a nonsensical misreading by moving the modifier closer to the word it modifies. Signed-off-by: brian m. carlson sand...@crustytoothpaste.net ---

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

2014-08-04 Thread brian m. carlson
Avoid a nonsensical misreading by removing needless text. Signed-off-by: brian m. carlson sand...@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

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

2014-08-04 Thread Jonathan Nieder
brian m. carlson wrote: --- 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

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

2014-08-04 Thread Jonathan Nieder
Jonathan Nieder wrote: Should the neighboring instances of '[Rr]equired to use imap-send be changed to plain Required, too? (I suspect yes.) Here's what that would look like. -- 8 -- From: brian m. carlson sand...@crustytoothpaste.net Subject: imap-send doc: omit confusing to use imap-send

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

2014-08-04 Thread brian m. carlson
On Mon, Aug 04, 2014 at 07:51:08PM -0700, Jonathan Nieder wrote: Jonathan Nieder wrote: Should the neighboring instances of '[Rr]equired to use imap-send be changed to plain Required, too? (I suspect yes.) Here's what that would look like. -- 8 -- From: brian m. carlson

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

2014-08-04 Thread Jonathan Nieder
brian m. carlson wrote: This has an extra period at the end of the line. Good catch. -- 8 -- Subject: fixup! imap-send doc: omit confusing to use imap-send modifier --- Thanks, Jonathan Documentation/git-imap-send.txt | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

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

2014-08-02 Thread brian m. carlson
Avoid a nonsensical misreading by moving the modifier closer to the word it modifies. Signed-off-by: brian m. carlson sand...@crustytoothpaste.net --- Documentation/git-imap-send.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-imap-send.txt