git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] notes: avoid empty line in template

From
Michael J Gruber <git@grubix.eu>
Date
Nov 16, 2022, 15:56 UTC
Message-ID
<347ee7ad6fea7cf96bb1e51772802102082b58cc.1668614158.git.git@grubix.eu>

When `git notes` prepares the template it adds an empty newline between the comment header and the content:

Show 6 quoted lines
>
> #
> # Write/edit the notes for the following object:
>
> # commit 0f3c55d4c2b7864bffb2d92278eff08d0b2e083f
> # etc

This is wrong structurally because that newline is part of the comment, too, and thus should be commented. Also, it throws off some positioning strategies of editors and plugins, and it differs from how we do commit templates.

Change this to follow the standard set by `git commit`:
Show 6 quoted lines
>
> #
> # Write/edit the notes for the following object:
> #
> # commit 0f3c55d4c2b7864bffb2d92278eff08d0b2e083f
>
Tests pass unchanged after this code change.
Signed-off-by: Michael J Gruber <git@grubix.eu>
---
 builtin/notes.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/notes.c b/builtin/notes.c
index be51f69225..80d9dfd25c 100644
--- a/builtin/notes.c
+++ b/builtin/notes.c
@@ -181,7 +181,7 @@ static void prepare_note_data(const struct object_id *object, struct note_data *
 		strbuf_addch(&buf, '\n');
 		strbuf_add_commented_lines(&buf, "\n", strlen("\n"));
 		strbuf_add_commented_lines(&buf, _(note_template), strlen(_(note_template)));
-		strbuf_addch(&buf, '\n');
+		strbuf_add_commented_lines(&buf, "\n", strlen("\n"));
 		write_or_die(fd, buf.buf, buf.len);
 
 		write_commented_object(fd, object);
-- 
2.38.1.672.gc8cd8f59d3
Next: Jeff King
Message 1 of 6 in “notes: avoid empty line in template”
  1. notes: avoid empty line in templateMichael J Gruber, Nov 16, 2022
  2. Jeff KingNov 16, 2022
  3. Taylor BlauNov 16, 2022
  4. Ævar Arnfjörð BjarmasonNov 17, 2022
  5. Michael J GruberNov 17, 2022
  6. Jeff KingNov 17, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.