* [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'
@ 2018-02-27 20:17 Mårten Kongstad
2018-02-27 20:24 ` Jonathan Nieder
0 siblings, 1 reply; 3+ messages in thread
From: Mårten Kongstad @ 2018-02-27 20:17 UTC (permalink / raw)
To: git; +Cc: Mårten Kongstad
Remove erroneous space between % and < in '% <(<N>)'.
Signed-off-by: Mårten Kongstad <marten.kongstad@gmail.com>
---
Documentation/pretty-formats.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt
index e664c08..6109ef0 100644
--- a/Documentation/pretty-formats.txt
+++ b/Documentation/pretty-formats.txt
@@ -202,7 +202,7 @@ endif::git-rev-list[]
- '%>>(<N>)', '%>>|(<N>)': similar to '%>(<N>)', '%>|(<N>)'
respectively, except that if the next placeholder takes more spaces
than given and there are spaces on its left, use those spaces
-- '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'
+- '%><(<N>)', '%><|(<N>)': similar to '%<(<N>)', '%<|(<N>)'
respectively, but padding both sides (i.e. the text is centered)
- %(trailers[:options]): display the trailers of the body as interpreted
by linkgit:git-interpret-trailers[1]. The `trailers` string may be
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'
2018-02-27 20:17 [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)' Mårten Kongstad
@ 2018-02-27 20:24 ` Jonathan Nieder
2018-02-28 9:39 ` Duy Nguyen
0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Nieder @ 2018-02-27 20:24 UTC (permalink / raw)
To: Mårten Kongstad; +Cc: git, Nguyễn Thái Ngọc Duy
Mårten Kongstad wrote:
> Remove erroneous space between % and < in '% <(<N>)'.
>
> Signed-off-by: Mårten Kongstad <marten.kongstad@gmail.com>
> ---
> Documentation/pretty-formats.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Looks correct to me. The space was introduced in v1.8.3-rc0~22^2
(pretty: support %>> that steals trailing spaces, 2013-04-19) and
appears to be a plain typo. Thanks for fixing it.
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Thanks.
> --- a/Documentation/pretty-formats.txt
> +++ b/Documentation/pretty-formats.txt
> @@ -202,7 +202,7 @@ endif::git-rev-list[]
> - '%>>(<N>)', '%>>|(<N>)': similar to '%>(<N>)', '%>|(<N>)'
> respectively, except that if the next placeholder takes more spaces
> than given and there are spaces on its left, use those spaces
> -- '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'
> +- '%><(<N>)', '%><|(<N>)': similar to '%<(<N>)', '%<|(<N>)'
> respectively, but padding both sides (i.e. the text is centered)
> - %(trailers[:options]): display the trailers of the body as interpreted
> by linkgit:git-interpret-trailers[1]. The `trailers` string may be
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'
2018-02-27 20:24 ` Jonathan Nieder
@ 2018-02-28 9:39 ` Duy Nguyen
0 siblings, 0 replies; 3+ messages in thread
From: Duy Nguyen @ 2018-02-28 9:39 UTC (permalink / raw)
To: Jonathan Nieder; +Cc: Mårten Kongstad, Git Mailing List
On Wed, Feb 28, 2018 at 3:24 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:
> Mårten Kongstad wrote:
>
>> Remove erroneous space between % and < in '% <(<N>)'.
>>
>> Signed-off-by: Mårten Kongstad <marten.kongstad@gmail.com>
>> ---
>> Documentation/pretty-formats.txt | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Looks correct to me. The space was introduced in v1.8.3-rc0~22^2
> (pretty: support %>> that steals trailing spaces, 2013-04-19) and
> appears to be a plain typo. Thanks for fixing it.
Yeah definitely a good change.
--
Duy
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-02-28 9:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-27 20:17 [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)' Mårten Kongstad
2018-02-27 20:24 ` Jonathan Nieder
2018-02-28 9:39 ` Duy Nguyen
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).