This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran 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 30437 [4.0/4.1/4.2/4.3 Regression] -Wno-all is rejected (try 2)

From: Brooks Moses <brooks dot moses at codesourcery dot com>
To: fortran at gcc dot gnu dot org
Cc: gcc-patches at gcc dot gnu dot org
Date: Wed, 17 Jan 2007 12:52:40 -0800
Subject: Re: PR 30437 [4.0/4.1/4.2/4.3 Regression] -Wno-all is rejected (try 2)
References: <6c33472e0701171140u1fada523r9bbcead04a0ac53e@mail.gmail.com>

Manuel López-Ibáñez wrote:

2007-01-17 Manuel Lopez-Ibanez <manu@gcc.gnu.org>

fortran/
  PR fortran/30437
  * lang.opt (Wall): Remove RejectNegative.
  * options.c (gfc_handle_option): Wall can be disabled.
  (set_Wall): Add a parameter for disabling Wall.

testsuite/
  PR fortran/30437
  * gcc.dg/Wall.c: New.
  * gcc.dg/Wno-all.c: New.
  * gfortran.dg/Wall.f90: New.
  * gfortran.dg/Wno-all.f90: New.

For the record, the Fortran parts of this look okay to me (though I'm not a Fortran maintainer, just someone who was recently poking about in options.c).

I specifically want to note that the handling of gfc_options.warn_tabs is correct, because at first glance it looks wrong. For some reason, the Fortran -Wtabs option is set up "backwards", so that -Wtabs turns the warning off, and -Wno-tabs turns it on, and thus it is indeed correct for -Wall to imply -Wno-tabs and vice-versa.

- Brooks

References:
- PR 30437 [4.0/4.1/4.2/4.3 Regression] -Wno-all is rejected (try 2)
  - From: Manuel López-Ibáñez

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