[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/3] grep: add --warnings={always,never,auto}.
From: |
Paul Eggert |
Subject: |
Re: [PATCH 1/3] grep: add --warnings={always,never,auto}. |
Date: |
Fri, 27 Aug 2010 11:38:05 -0700 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.11) Gecko/20100713 Thunderbird/3.0.6 |
On 08/27/2010 03:56 AM, 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.
I don't understand that argument. Is it saying that because we're
already violating POSIX unless POSIXLY_CORRECT is set,
that means it's OK to violate the GNU coding standards unconditionally?
I'm with Jim here. Let's disable the warnings if POSIXLY_CORRECT
(because POSIX says so), but leave the warnings in otherwise
(because the GNU coding standards say so).
- [PATCH 1/3] grep: add --warnings={always,never,auto}., Paolo Bonzini, 2010/08/15
- [PATCH 2/3] dfa: warn on [:space:] and similar, Paolo Bonzini, 2010/08/15
- [PATCH 3/3] tests: add test for warnings, Paolo Bonzini, 2010/08/15
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Jim Meyering, 2010/08/27
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Paolo Bonzini, 2010/08/27
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}.,
Paul Eggert <=
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Paolo Bonzini, 2010/08/30
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Paul Eggert, 2010/08/30
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Paolo Bonzini, 2010/08/31
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Jim Meyering, 2010/08/31
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Paolo Bonzini, 2010/08/31
- Re: [PATCH 1/3] grep: add --warnings={always,never,auto}., Jim Meyering, 2010/08/27