[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#17472: [PATCH] dfa.c (lex): Remove capitalisation of two dfaerror st

From: Jim Meyering
Subject: bug#17472: [PATCH] dfa.c (lex): Remove capitalisation of two dfaerror strings
Date: Fri, 23 May 2014 22:27:10 -0700

On Sun, May 11, 2014 at 11:12 PM, Jim Meyering <address@hidden> wrote:
> On Sun, May 11, 2014 at 9:01 PM, behoffski <address@hidden> wrote:
>> A few months ago, "make syntax-check" would complain if any string that
>> was subject to localisation started with an upper-case character.
>> Almost all messages in Grep conform to this standard.  However,
>> "make syntax-check" no longer complains about two cases in the current
>> master (dfa.c, function lex ()):
>>     dfaerror (_("Invalid content of \\{\\}"));
>>     dfaerror (_("Regular expression too big"));
>> I don't know if this constraint is being relaxed, or if the check has
>> changed somehow.  (Is syntax-check provided via gnulib?)
>> Attached is a simple patch that converts the two error strings above to
>> have a lower-case first character.  This may be useful to help maintain
>> consistent message formatting.
> Thanks for the patch.
> That syntax-check rule does indeed come from gnulib, but it
> looks only at error-like functions whose names match /[^rp]error/.
> That hard-coded pattern would better be customizable.
> BTW, those strings were added in 2012.
> I've adjusted the commit message to be more consistent, and will
> defer pushing your change to master until after the release.
> Many translation teams have already updated strings for the
> release, so I'd rather not change translatable messages
> until after 2.19.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]