This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: warning message control: jumbo patch so far
- From: Gabriel Dos Reis <gdr at integrable-solutions dot net>
- To: DJ Delorie <dj at redhat dot com>
- Cc: mark at codesourcery dot com, gcc-patches at gcc dot gnu dot org
- Date: 03 May 2005 23:38:26 +0200
- Subject: Re: warning message control: jumbo patch so far
- References: <200504230001.j3N01RR8016620@greed.delorie.com><4269966E.1020708@codesourcery.com><200504230039.j3N0dWvp017120@greed.delorie.com><4269A6B7.7070402@codesourcery.com><200504232138.j3NLcscP001625@greed.delorie.com><200504252153.j3PLr5X9026266@greed.delorie.com><200504292255.j3TMtSFY013487@greed.delorie.com><m3k6mkjtai.fsf@uniton.integrable-solutions.net><200505022127.j42LRpgL001732@greed.delorie.com><m34qdkahyg.fsf@uniton.integrable-solutions.net><200505031830.j43IUHJH003431@greed.delorie.com>
DJ Delorie <dj@redhat.com> writes:
| > follow up with a patch
|
| > * that adds a field "show_options" to diagnostic_context, that
| > consistently play the role of diagnostic_show_options;
| > * that has this
| >
| > | + if (opt && ! option_enabled (opt))
| > | + return;
| > | +
| > done by report_diagnostic. That for example,
| > ensures that any diagnostic kind (error, warning, note, ...) has
| > it checked properly.
|
| Well, one of our customers wants to be able to silence certain
| pedantic errors, so I'm working in this direction anyway ;-)
great!
| pedwarn() will eventually have to have the extra option_index
| parameter too, perhaps error() et al also.
Yes, understood. When those are properly converted, then
error(), warning(), pedwarn() would just be syntactic convenience over
report_diagnostic().
-- Gaby