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 6:06 PM, Manuel López-Ibáñez
<lopezibanez@gmail.com> wrote:
> On 18 April 2010 00:03, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
>
>>>> ?(2) issue the [...] note only once.
>>>
>>> That is too complex to implement.
>>
>> Hmm, it's not possible to keep a bitmap of OPT_W_blah values, or
>> something like that?
>
> One option OPT_W* may control different text messages. So, it has to
> be attached to each different text message.

That is a vastly different thing that what I have been talking about since the
beginning.  I have no problem with your objecting that.  But that objection
should be understood as irrelevant to what I have been saying:  The default
value of fdiagnostics-show-option should be inferred from configuration options.

>> Anyway, I think your patch is a good thing, even if the note issued
>> more than once.
>
> That depends how you understand "more than once". It is attached to
> each diagnostic. I understand that what Gabriel proposes is to attach
> it only to the first diagnostic with the same text.

That is correct.

-- Gaby


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