# [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'

3 messages from 2018-02-27 to 2018-02-28. Participants: Mårten Kongstad, Jonathan Nieder, Duy Nguyen.
Thread: https://gitlist.dev/t/47935

## Mårten Kongstad, 2018-02-27 20:17

Subject: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'
Message-ID: <1519762664-16889-1-git-send-email-marten.kongstad@gmail.com>
URL: https://gitlist.dev/e/1519762664-16889-1-git-send-email-marten.kongstad%40gmail.com

```
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


```

## Jonathan Nieder, 2018-02-27 20:24

Subject: Re: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'
Message-ID: <20180227202423.GD174036@aiede.svl.corp.google.com>
URL: https://gitlist.dev/e/20180227202423.GD174036%40aiede.svl.corp.google.com
In-Reply-To: <1519762664-16889-1-git-send-email-marten.kongstad@gmail.com>

```
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

```

## Duy Nguyen, 2018-02-28 09:39

Subject: Re: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'
Message-ID: <CACsJy8CnxYyH2-diTXJXxkNH5i8MkpgK+6xQ9yrXro+bc_bgfg@mail.gmail.com>
URL: https://gitlist.dev/e/CACsJy8CnxYyH2-diTXJXxkNH5i8MkpgK%2B6xQ9yrXro%2Bbc_bgfg%40mail.gmail.com
In-Reply-To: <20180227202423.GD174036@aiede.svl.corp.google.com>

```
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

```