{"thread":{"id":"47935","subject":"[PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'","startedAt":"2018-02-27T20:18:14Z","lastAt":"2018-02-28T09:40:22Z","messageCount":3,"participants":["Mårten Kongstad","Jonathan Nieder","Duy Nguyen"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"340440","messageId":"1519762664-16889-1-git-send-email-marten.kongstad@gmail.com","threadId":"47935","inReplyTo":null,"subject":"[PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'","fromName":"Mårten Kongstad","fromEmail":"marten.kongstad@gmail.com","sentAt":"2018-02-27T20:17:44Z","receivedAt":"2018-02-27T20:18:14Z","isPatch":true,"sender":{"key":"marten.kongstad@gmail.com","avatar":"https://avatars.githubusercontent.com/u/567558?v=4"},"body":"Remove erroneous space between % and < in '% <(<N>)'.\n\nSigned-off-by: Mårten Kongstad <marten.kongstad@gmail.com>\n---\n Documentation/pretty-formats.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt\nindex e664c08..6109ef0 100644\n--- a/Documentation/pretty-formats.txt\n+++ b/Documentation/pretty-formats.txt\n@@ -202,7 +202,7 @@ endif::git-rev-list[]\n - '%>>(<N>)', '%>>|(<N>)': similar to '%>(<N>)', '%>|(<N>)'\n   respectively, except that if the next placeholder takes more spaces\n   than given and there are spaces on its left, use those spaces\n-- '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'\n+- '%><(<N>)', '%><|(<N>)': similar to '%<(<N>)', '%<|(<N>)'\n   respectively, but padding both sides (i.e. the text is centered)\n - %(trailers[:options]): display the trailers of the body as interpreted\n   by linkgit:git-interpret-trailers[1]. The `trailers` string may be\n-- \n1.9.1\n\n"},{"id":"340441","messageId":"20180227202423.GD174036@aiede.svl.corp.google.com","threadId":"47935","inReplyTo":"1519762664-16889-1-git-send-email-marten.kongstad@gmail.com","subject":"Re: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2018-02-27T20:24:23Z","receivedAt":"2018-02-27T20:24:30Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Mårten Kongstad wrote:\n\n> Remove erroneous space between % and < in '% <(<N>)'.\n>\n> Signed-off-by: Mårten Kongstad <marten.kongstad@gmail.com>\n> ---\n>  Documentation/pretty-formats.txt | 2 +-\n>  1 file changed, 1 insertion(+), 1 deletion(-)\n\nLooks correct to me.  The space was introduced in v1.8.3-rc0~22^2\n(pretty: support %>> that steals trailing spaces, 2013-04-19) and\nappears to be a plain typo.  Thanks for fixing it.\n\nReviewed-by: Jonathan Nieder <jrnieder@gmail.com>\n\nThanks.\n\n> --- a/Documentation/pretty-formats.txt\n> +++ b/Documentation/pretty-formats.txt\n> @@ -202,7 +202,7 @@ endif::git-rev-list[]\n>  - '%>>(<N>)', '%>>|(<N>)': similar to '%>(<N>)', '%>|(<N>)'\n>    respectively, except that if the next placeholder takes more spaces\n>    than given and there are spaces on its left, use those spaces\n> -- '%><(<N>)', '%><|(<N>)': similar to '% <(<N>)', '%<|(<N>)'\n> +- '%><(<N>)', '%><|(<N>)': similar to '%<(<N>)', '%<|(<N>)'\n>    respectively, but padding both sides (i.e. the text is centered)\n>  - %(trailers[:options]): display the trailers of the body as interpreted\n>    by linkgit:git-interpret-trailers[1]. The `trailers` string may be\n"},{"id":"340548","messageId":"CACsJy8CnxYyH2-diTXJXxkNH5i8MkpgK+6xQ9yrXro+bc_bgfg@mail.gmail.com","threadId":"47935","inReplyTo":"20180227202423.GD174036@aiede.svl.corp.google.com","subject":"Re: [PATCH] docs/pretty-formats: fix typo '% <(<N>)' -> '%<|(<N>)'","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2018-02-28T09:39:47Z","receivedAt":"2018-02-28T09:40:22Z","isPatch":true,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Wed, Feb 28, 2018 at 3:24 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:\n> Mårten Kongstad wrote:\n>\n>> Remove erroneous space between % and < in '% <(<N>)'.\n>>\n>> Signed-off-by: Mårten Kongstad <marten.kongstad@gmail.com>\n>> ---\n>>  Documentation/pretty-formats.txt | 2 +-\n>>  1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> Looks correct to me.  The space was introduced in v1.8.3-rc0~22^2\n> (pretty: support %>> that steals trailing spaces, 2013-04-19) and\n> appears to be a plain typo.  Thanks for fixing it.\n\nYeah definitely a good change.\n-- \nDuy\n"}]}