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: [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context

From: Jeff Law <law at redhat dot com>
To: Jason Merrill <jason at redhat dot com>, Bernd Edlinger <bernd dot edlinger at hotmail dot de>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Joseph Myers <joseph at codesourcery dot com>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>
Date: Thu, 15 Sep 2016 09:44:40 -0600
Subject: Re: [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context
Authentication-results: sourceware.org; auth=none
References: <AM4PR0701MB21624AB3DECA1269E4BBB475E4E50@AM4PR0701MB2162.eurprd07.prod.outlook.com> <4fd68972-b48b-560a-b8e2-ae7d607b9b87@redhat.com> <AM4PR0701MB216265F6259DAEEFB6C52AA0E4FF0@AM4PR0701MB2162.eurprd07.prod.outlook.com> <48602fe1-c295-fce5-a6fe-e7e259532f44@redhat.com> <AM4PR0701MB2162C05959D824E57DDC6B12E4FF0@AM4PR0701MB2162.eurprd07.prod.outlook.com> <AM4PR0701MB2162918E947B0A81ED74C940E4F10@AM4PR0701MB2162.eurprd07.prod.outlook.com> <CADzB+2kTxpVQbW_AFSwbxmn3r5bVRT93wYaRpnanV5FoT7y_kQ@mail.gmail.com> <AM4PR0701MB2162863B98AD755E0A9AC189E4F10@AM4PR0701MB2162.eurprd07.prod.outlook.com> <CADzB+2kw17THX1i_2X2m1EHxScOOFF3vbVevYoPDZDUkwQzcGQ@mail.gmail.com>

On 09/14/2016 12:11 PM, Jason Merrill wrote:


I think we could have both, where the weaker form is part of -Wall and
people can explicitly select the stronger form.

That's been a fairly standard way to handle this kind of thing. Itworks for me.


Jeff

Follow-Ups:
- Re: [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context
  - From: Bernd Edlinger

References:
- [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context
  - From: Bernd Edlinger
- Re: [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context
  - From: Jason Merrill
- Re: [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context
  - From: Bernd Edlinger
- Re: [PATCH, updated] Add a warning for suspicious use of conditional expressions in boolean context
  - From: Jason Merrill

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