This is the mail archive of the 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 18 April 2010 00:03, Steven Bosscher <> 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.

Even if each unique call to warning had its own bit, I find confusing to say:

warning: 'd' is uninitialized [-Wuninitialized]
[...loads of other stuff scrolling the above line up...]
warning: 'j' is uninitialized
warning: 'z' may be uninitialized [-Wuninitialized]

If you think the above is nice and trivial to implement, go ahead.

> 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.



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