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: [patch] unify free- and fixed-form diagnostic messages

From: Bernhard Fischer <rep dot nop at aon dot at>
To: Jerry DeLisle <jvdelisle at verizon dot net>
Cc: fortran at gcc dot gnu dot org, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 22 Mar 2006 12:17:59 +0100
Subject: Re: [patch] unify free- and fixed-form diagnostic messages
References: <20060321170323.GB5394@aon.at> <4420D5AA.4090905@verizon.net>

On Tue, Mar 21, 2006 at 08:42:18PM -0800, Jerry DeLisle wrote:
>Bernhard Fischer wrote:
>>Hi,
>>
>>Attached patch makes two warning messages consistent between fixed-form
>>and free-form.
>>
>>Ok for trunk and 4.1?

2006-03-21  Bernhard Fischer  <aldot@gcc.gnu.org>

	* parse.c (next_free): use consistent error string between
	free-form and fixed-form for illegal statement label of zero.
	(next_fixed): use consistent warning string between free-form
	and fixed-form for statement labels for empty statements.

2006-03-22  Bernhard Fischer  <aldot@gcc.gnu.org>

	* label_1.f90: Adjust dg-error.

>>
>Hi Bernard,
>
>On first glance this looks OK.  Have you regression tested to make sure we 
>do not have any dg-error checks in the testsuite that trigger off these 
>error messages? (I have not tested yet)

I forgot to include the diff to label_1.f90, sorry. Updated patch which
also corrects the label_1.f90 testcase is attached.
No, i didn't regression-test it.

Attachment: gcc.gfortran.unify_freefixed_msg.02.diff
Description: Text document

References:
- Re: [patch] unify free- and fixed-form diagnostic messages
  - From: Jerry DeLisle

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