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: [C++ Patch] PR 50810 (new try)

From: Paolo Carlini <paolo dot carlini at oracle dot com>
To: Gabriel Dos Reis <gdr at integrable-solutions dot net>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Jason Merrill <jason at redhat dot com>
Date: Mon, 24 Oct 2011 14:18:50 +0200
Subject: Re: [C++ Patch] PR 50810 (new try)
References: <4EA5503C.7070202@oracle.com> <CAAiZkiA2rrsfH=Q9GmwxK+mcOFjU=h6B+oQBZ_aBGfn=Cs=HwQ@mail.gmail.com>

Hi,

On Mon, Oct 24, 2011 at 6:47 AM, Paolo Carlini<paolo.carlini@oracle.com> wrote:

Hi,

the below is a new variant removing -Wc++0x-compat from -Wall (cannot be
added to -Wextra either because bootstrap passes -W) and also, as requested
by Gaby, preventing -Wno-narrowing from suppressing the warning in C++0x
mode (if the user really needs to silence it, -Wno-c++0x-compat works). I
also added a new testcase for that.

OK with a minor correction. This bit

+With -std=c++0x, @option{-Wno-c++0x-compat} can be used to suppress
+the diagnostic required by the standard.

should not be there.  It is currently an accident of implementation
detail as opposed to a feature.  It needs no advertisement.

Ok. But I actively made it possible, if you want I can remove the possibility altogether, the patch also becomes cleaner ;)

Paolo.

Follow-Ups:
- Re: [C++ Patch] PR 50810 (new try)
  - From: Paolo Carlini
- Re: [C++ Patch] PR 50810 (new try)
  - From: Gabriel Dos Reis

References:
- [C++ Patch] PR 50810 (new try)
  - From: Paolo Carlini
- Re: [C++ Patch] PR 50810 (new try)
  - From: Gabriel Dos Reis

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