Re: [patch] Clarify interaction of -Wnarrowing with -std

2017-03-27 Thread Jonathan Wakely
On 27/03/17 10:09 +0100, Jonathan Wakely wrote: On 26/03/17 14:32 -0600, Sandra Loosemore wrote: On 03/26/2017 02:13 PM, Gerald Pfeifer wrote: Hi Jonathan, On Tue, 23 Feb 2016, Jonathan Wakely wrote: On 19/02/16 13:17 -0700, Sandra Loosemore wrote: I suppose the patch is OK as it stands, but

Re: [patch] Clarify interaction of -Wnarrowing with -std

2017-03-27 Thread Jonathan Wakely
On 26/03/17 14:32 -0600, Sandra Loosemore wrote: On 03/26/2017 02:13 PM, Gerald Pfeifer wrote: Hi Jonathan, On Tue, 23 Feb 2016, Jonathan Wakely wrote: On 19/02/16 13:17 -0700, Sandra Loosemore wrote: I suppose the patch is OK as it stands, but I was going to suggest restructuring it so that

Re: [patch] Clarify interaction of -Wnarrowing with -std

2017-03-26 Thread Sandra Loosemore
On 03/26/2017 02:13 PM, Gerald Pfeifer wrote: Hi Jonathan, On Tue, 23 Feb 2016, Jonathan Wakely wrote: On 19/02/16 13:17 -0700, Sandra Loosemore wrote: I suppose the patch is OK as it stands, but I was going to suggest restructuring it so that it talks about the default behavior first and what

Re: [patch] Clarify interaction of -Wnarrowing with -std

2017-03-26 Thread Gerald Pfeifer
Hi Jonathan, On Tue, 23 Feb 2016, Jonathan Wakely wrote: On 19/02/16 13:17 -0700, Sandra Loosemore wrote: I suppose the patch is OK as it stands, but I was going to suggest restructuring it so that it talks about the default behavior first and what it does with non-default -std= options after t

Re: [patch] Clarify interaction of -Wnarrowing with -std

2016-02-23 Thread Jonathan Wakely
On 19/02/16 13:17 -0700, Sandra Loosemore wrote: On 02/19/2016 12:01 PM, Jason Merrill wrote: On 02/19/2016 07:42 AM, Jonathan Wakely wrote: In PR69864 Manu suggests improving the docs to explain that -Wnarrowing sometimes produces errors not warnings. I think the right way to do that is clari

Re: [patch] Clarify interaction of -Wnarrowing with -std

2016-02-19 Thread Jonathan Wakely
On 19/02/16 13:17 -0700, Sandra Loosemore wrote: On 02/19/2016 12:01 PM, Jason Merrill wrote: On 02/19/2016 07:42 AM, Jonathan Wakely wrote: In PR69864 Manu suggests improving the docs to explain that -Wnarrowing sometimes produces errors not warnings. I think the right way to do that is clari

Re: [patch] Clarify interaction of -Wnarrowing with -std

2016-02-19 Thread Sandra Loosemore
On 02/19/2016 12:01 PM, Jason Merrill wrote: On 02/19/2016 07:42 AM, Jonathan Wakely wrote: In PR69864 Manu suggests improving the docs to explain that -Wnarrowing sometimes produces errors not warnings. I think the right way to do that is clarify how it interacts with -std. Specifically that t

Re: [patch] Clarify interaction of -Wnarrowing with -std

2016-02-19 Thread Jason Merrill
On 02/19/2016 07:42 AM, Jonathan Wakely wrote: In PR69864 Manu suggests improving the docs to explain that -Wnarrowing sometimes produces errors not warnings. I think the right way to do that is clarify how it interacts with -std. Specifically that the effect of -Wnarrowing listed first in the m

[patch] Clarify interaction of -Wnarrowing with -std

2016-02-19 Thread Jonathan Wakely
In PR69864 Manu suggests improving the docs to explain that -Wnarrowing sometimes produces errors not warnings. I think the right way to do that is clarify how it interacts with -std. Specifically that the effect of -Wnarrowing listed first in the manual *only* applies to C++98 modes, For all lat