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: PR 30824 -Werror -Wfatal-errors should stop after the first warning


On Mon, 19 Feb 2007, Manuel López-Ibáñez wrote:

| On 19 Feb 2007 14:06:19 -0600, Gabriel Dos Reis <gdr@cs.tamu.edu> wrote:
| > "Manuel López-Ibáñez" <lopezibanez@gmail.com> writes:
| >
| > | Notable differences in this patch from DJ's version are:
| > |
| > | * The message of -Wfatal-errors now prints the program name as most
| > | other messages do. "cc1: compilation terminated due to
| > | -Wfatal-errors.". I needed to do this in order to be able to construct
| > | testcases.
| >
| > Hmm, I failed to see that need in the testcases you submitted.  Could you
| > elaborate?
| >
|
| dg-warning, dg-error were not able to match the message otherwise.
| Perhaps they expect a ":".  Humm, now that I look to it more closely,
| I think that dejagnu expects at least one character before the
| pattern, so I could change the pattern to something like "terminated
| due to -Wfatal-errors" and it may work.

yes, please try that one.

|
| In any case, many messages print the program name (for example,
| "warnings treated as errors" does), so I thought it was consistent.
|
| Let me know if you really hate it and I can try changing the matching pattern.

In this specific case I think we should leave out the 'cc1' part.
I cannot remember if it is OK (GNU standard) to put the last period --
I believe it is not.

-- Gaby


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