Re: [PATCH 1/3] grep: add --warnings={always,never,auto}.

[Jim Meyering]

Paolo Bonzini wrote:

> On 08/27/2010 12:13 PM, Jim Meyering wrote:
>>      I have a fundamental aversion to making functionality like this new
>>      warning depend on whether a standard input or output stream is a TTY.
>>      Personally, I want my script to fail if I use a regexp that deserves
>>      a warning.
>
> Me too, but this would be against POSIX, and I think it is also
> borderline to remove the warning only for POSIXLY_CORRECT.  But I
> agree we can say in NEWS that the default for --warn may be changed in
> the future, just in case.
>
>> -    dfawarn ("character classes syntax is [[:space:]], not [:space:]");
>> +    dfawarn ("character class syntax is [[:space:]], not [:space:]");
>
> You're the native speaker, so I'm not going to complain. :)
>
> You're welcome to push the amended patchset!

Thanks.  One final change was to mark the dfa_warn argument
for translation:

    dfawarn (_("character class syntax is [[:space:]], not [:space:]"));

Now, I've pushed it.
We can adjust semantics separately.