A Seg, 25-07-2016 às 10:49 -0700, Junio C Hamano escreveu:
> Vasco Almeida <vascomalme...@sapo.pt> writes:
> 
> > 
> >  static const char note_template[] =
> > -   "\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,8 @@ 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_addch(&buf, '\n');
> > +           strbuf_add_commented_lines(&buf, _(note_template),
> > strlen(_(note_template)));
> 
> I do not quite understand why you want the blank lines surrounding
> the message outside add_commented_lines() call.  I think the intent
> is to produce
> 
>     #
>     # Write/edit the notes for the following object:
>     #

If this is what we want, I will send a re-roll accordingly.

> with the single call.  If you pushed the newlines outside the
> message, wouldn't you end up having this instead (____ denoting an
> extra empty line each before and after the message)?
> 
>     ____
>     # Write/edit the notes for the following object:
>     ____
> 
Yes, this was my intention. The original does:

    #
    # Write/edit the notes for the following object:
    ____

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to