This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: enable fdiagnostics-show-option by default


On Sat, Apr 17, 2010 at 12:29 PM, Ian Lance Taylor <iant@google.com> wrote:
> Manuel López-Ibáñez <lopezibanez@gmail.com> writes:
>
>> On 16 April 2010 16:01, Ian Lance Taylor <iant@google.com> wrote:
>>>>
>>>> Why not? ?After all, the patch is changing the current default and users
>>>> who want to have the old default would have to do it in a user-by-user basis.
>>>
>>> Because I believe that in general it serves the community better for
>>> all naïve users to see the same presentation for warning and error
>>> messages, modulo translation.
>>>
>>> Also because there are hundreds of aspects of gcc behaviour that could
>>> in principle be controlled at configuration time. ?Why single this one
>>> out? ?gcc is already too configurable as it is. ?Too much
>>> configurability is not a feature. ?It adds complexity to the overall
>>> system for minimal gain.
>>
>> So I guess a patch to add a configure time switch won't be accepted.
>> It is quite a pain in the ass to write it, so I don't want to waste
>> (more) my time. Is this a deadlock? No solution?
>
> So far I think Gaby is opposed to your proposed change and Diego and I
> are in favor of it.

More accurately, I'm opposed to the patch in its current form.  I offered
two alternatives:
  (1) make the behaviour configurable at configure-time
  (2) issue the [...] note only once.

-- Gaby


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]