# [PATCH] i18n: notes: mark comment for translation

2 messages from 2016-07-21 to 2016-07-21. Participants: Vasco Almeida, Ævar Arnfjörð Bjarmason.
Thread: https://gitlist.dev/t/42898

## Vasco Almeida, 2016-07-21 10:07

Subject: [PATCH] i18n: notes: mark comment for translation
Message-ID: <1469095670-3834-1-git-send-email-vascomalmeida@sapo.pt>
URL: https://gitlist.dev/e/1469095670-3834-1-git-send-email-vascomalmeida%40sapo.pt

```
Mark comment displayed when editing a note for translation.

Signed-off-by: Vasco Almeida <vascomalmeida@sapo.pt>
---
 builtin/notes.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/builtin/notes.c b/builtin/notes.c
index 0572051..37933bd 100644
--- a/builtin/notes.c
+++ b/builtin/notes.c
@@ -91,7 +91,7 @@ static const char * const git_notes_get_ref_usage[] = {
 };
 
 static const char note_template[] =
-	"\nWrite/edit the notes for the following object:\n";
+	N_("\nWrite/edit the notes for the following object:\n");
 
 struct note_data {
 	int given;
@@ -179,7 +179,7 @@ static void prepare_note_data(const unsigned char *object, struct note_data *d,
 			copy_obj_to_fd(fd, old_note);
 
 		strbuf_addch(&buf, '\n');
-		strbuf_add_commented_lines(&buf, note_template, strlen(note_template));
+		strbuf_add_commented_lines(&buf, _(note_template), strlen(_(note_template)));
 		strbuf_addch(&buf, '\n');
 		write_or_die(fd, buf.buf, buf.len);
 
-- 
2.7.4


```

## Ævar Arnfjörð Bjarmason, 2016-07-21 10:45

Subject: Re: [PATCH] i18n: notes: mark comment for translation
Message-ID: <CACBZZX51K=D2gYbzKvJW3B5=bzxEGGFhOc6TkAfJkkpdYuEt0w@mail.gmail.com>
URL: https://gitlist.dev/e/CACBZZX51K%3DD2gYbzKvJW3B5%3DbzxEGGFhOc6TkAfJkkpdYuEt0w%40mail.gmail.com
In-Reply-To: <1469095670-3834-1-git-send-email-vascomalmeida@sapo.pt>

```
On Thu, Jul 21, 2016 at 12:07 PM, Vasco Almeida <vascomalmeida@sapo.pt> wrote:
> Mark comment displayed when editing a note for translation.

I'm not sure, but wouldn't this make more sense:

> Signed-off-by: Vasco Almeida <vascomalmeida@sapo.pt>
> ---
>  builtin/notes.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/builtin/notes.c b/builtin/notes.c
> index 0572051..37933bd 100644
> --- a/builtin/notes.c
> +++ b/builtin/notes.c
> @@ -91,7 +91,7 @@ static const char * const git_notes_get_ref_usage[] = {
>  };
>
>  static const char note_template[] =
> -       "\nWrite/edit the notes for the following object:\n";
> +       N_("\nWrite/edit the notes for the following object:\n");
+       N_("Write/edit the notes for the following object:");
>
>  struct note_data {
>         int given;
> @@ -179,7 +179,7 @@ static void prepare_note_data(const unsigned char *object, struct note_data *d,
>                         copy_obj_to_fd(fd, old_note);
>
>                 strbuf_addch(&buf, '\n');
+                 strbuf_addch(&buf, '\n');
> -               strbuf_add_commented_lines(&buf, note_template, strlen(note_template));
> +               strbuf_add_commented_lines(&buf, _(note_template), strlen(_(note_template)));
+                 strbuf_addch(&buf, '\n');
>                 strbuf_addch(&buf, '\n');
>                 write_or_die(fd, buf.buf, buf.len);

I.e. moving the starting and ending \n out of the message itself, the
surrounding whitespace doesn't seem like something the translators
need to modify, and if they were to trim it it looks like it would
break the output a bit.

```
