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

["Ævar Arnfjörð Bjarmason" <avarab@gmail.com>]

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.