Hi Junio,
On Sun, 11 Sep 2016, Junio C Hamano wrote:
> Johannes Schindelin <johannes.schindelin@xxxxxx> writes:
>
> > This makes the code consistent by fixing quite a couple of error messages.
>
> Looks OK. While at it, we may want another one to downcase the
> first word, perhaps?
>
> These may not be messages added by your series and can be left
> outside this series, but I have to point out that
>
> if (commit_lock_file(&msg_file) < 0)
> return error(_("Error wrapping up '%s'."), filename);
>
> results in "error: Error wrapping up", which sounds quite funny.
>
> "failed to finalize" or something would flow a bit better, I'd say.
Fair enough. I added a patch to make it so.
Thanks,
Dscho