{"thread":{"id":"12153","subject":"[PATCH 3/4] Retain extra blank lines between the summary and the body","startedAt":"2008-02-17T18:35:51Z","lastAt":"2008-02-17T18:35:51Z","messageCount":1,"participants":["Daniel Barkalow"],"isPatch":true,"patchVersion":1,"patchTotal":4},"messages":[{"id":"69017","messageId":"alpine.LNX.1.00.0802171335490.5816@iabervon.org","threadId":"12153","inReplyTo":null,"subject":"[PATCH 3/4] Retain extra blank lines between the summary and the body","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2008-02-17T18:35:51Z","receivedAt":"2008-02-17T18:35:51Z","isPatch":true,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"This is so that it's possible to have a message that will generate an\nemail with the first line blank. If the email you're generating is\nactually a template, you may have a single line filling in for a\n~3-line paragraph, meaning that the space starts with a blank line,\nwhich was impossible before as output from the message-splitting code.\n\nFurthermore, extra blank lines that aren't at the beginning of the\nbody portion aren't stripped, so this is more consistant overall.\n\nSigned-off-by: Daniel Barkalow <barkalow@iabervon.org>\n---\n pretty.c |    4 ----\n 1 files changed, 0 insertions(+), 4 deletions(-)\n\ndiff --git a/pretty.c b/pretty.c\nindex f82380b..98efac2 100644\n--- a/pretty.c\n+++ b/pretty.c\n@@ -713,7 +713,6 @@ void pp_remainder(enum cmit_fmt fmt,\n \t\t  struct strbuf *sb,\n \t\t  int indent)\n {\n-\tint first = 1;\n \tfor (;;) {\n \t\tconst char *line = *msg_p;\n \t\tint linelen = get_one_line(line);\n@@ -723,12 +722,9 @@ void pp_remainder(enum cmit_fmt fmt,\n \t\t\tbreak;\n \n \t\tif (is_empty_line(line, &linelen)) {\n-\t\t\tif (first)\n-\t\t\t\tcontinue;\n \t\t\tif (fmt == CMIT_FMT_SHORT)\n \t\t\t\tbreak;\n \t\t}\n-\t\tfirst = 0;\n \n \t\tstrbuf_grow(sb, linelen + indent + 20);\n \t\tif (indent) {\n-- \n1.5.4.1.1350.g2b9ee\n"}]}