{"thread":{"id":"46186","subject":"[PATCH] docs/pretty-formats: stress that %- removes all preceding line-feeds","startedAt":"2017-06-15T10:36:18Z","lastAt":"2017-06-16T18:30:43Z","messageCount":2,"participants":["SZEDER Gábor","Jonathan Nieder"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"322307","messageId":"20170615103607.11119-1-szeder.dev@gmail.com","threadId":"46186","inReplyTo":null,"subject":"[PATCH] docs/pretty-formats: stress that %- removes all preceding line-feeds","fromName":"SZEDER Gábor","fromEmail":"szeder.dev@gmail.com","sentAt":"2017-06-15T10:36:07Z","receivedAt":"2017-06-15T10:36:18Z","isPatch":true,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>\n---\n\nA mere plural \"line-feeds\" was too subtle for me to grasp on first\n(and second...) reading.\n\n Documentation/pretty-formats.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt\nindex 38040e95b..a48d267e2 100644\n--- a/Documentation/pretty-formats.txt\n+++ b/Documentation/pretty-formats.txt\n@@ -213,8 +213,8 @@ If you add a `+` (plus sign) after '%' of a placeholder, a line-feed\n is inserted immediately before the expansion if and only if the\n placeholder expands to a non-empty string.\n \n-If you add a `-` (minus sign) after '%' of a placeholder, line-feeds that\n-immediately precede the expansion are deleted if and only if the\n+If you add a `-` (minus sign) after '%' of a placeholder, all consecutive\n+line-feeds immediately preceding the expansion are deleted if and only if the\n placeholder expands to an empty string.\n \n If you add a ` ` (space) after '%' of a placeholder, a space\n-- \n2.13.1.505.g7cc9fcafb\n\n"},{"id":"322472","messageId":"20170616183034.GY133952@aiede.mtv.corp.google.com","threadId":"46186","inReplyTo":"20170615103607.11119-1-szeder.dev@gmail.com","subject":"Re: [PATCH] docs/pretty-formats: stress that %- removes all preceding line-feeds","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2017-06-16T18:30:34Z","receivedAt":"2017-06-16T18:30:43Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"SZEDER Gábor wrote:\n\n> Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>\n> ---\n> A mere plural \"line-feeds\" was too subtle for me to grasp on first\n> (and second...) reading.\n\nThis could go in the commit message.\n\n>  Documentation/pretty-formats.txt | 4 ++--\n>  1 file changed, 2 insertions(+), 2 deletions(-)\n\nWith or without such a tweak,\nReviewed-by: Jonathan Nieder <jrnieder@gmail.com>\n\nThanks.\n\n> diff --git a/Documentation/pretty-formats.txt b/Documentation/pretty-formats.txt\n> index 38040e95b..a48d267e2 100644\n> --- a/Documentation/pretty-formats.txt\n> +++ b/Documentation/pretty-formats.txt\n> @@ -213,8 +213,8 @@ If you add a `+` (plus sign) after '%' of a placeholder, a line-feed\n>  is inserted immediately before the expansion if and only if the\n>  placeholder expands to a non-empty string.\n>  \n> -If you add a `-` (minus sign) after '%' of a placeholder, line-feeds that\n> -immediately precede the expansion are deleted if and only if the\n> +If you add a `-` (minus sign) after '%' of a placeholder, all consecutive\n> +line-feeds immediately preceding the expansion are deleted if and only if the\n>  placeholder expands to an empty string.\n>  \n>  If you add a ` ` (space) after '%' of a placeholder, a space\n"}]}