Re: [C++ Patch] PR 65323

2015-03-12 Thread Jason Merrill
On 03/12/2015 06:13 AM, Paolo Carlini wrote: 52718_red.C:1:22: warning: zero as null pointer constant [-Wzero-as-null-pointer-constant] void* fun(void* a = 0); 52718_red.C:2:16: warning: zero as null pointer constant [-Wzero-as-null-pointer-constant] void* f2 = fun(); OK, then your second

Re: [C++ Patch] PR 65323

2015-03-12 Thread Paolo Carlini
Hi, On 03/11/2015 09:26 PM, Jason Merrill wrote: On 03/06/2015 03:36 AM, Paolo Carlini wrote: this is a regression about duplicate warnings with -Wzero-as-null-pointer-constant. The regression is rather old, affects 4_8-branch too, and started when check_default_argument got a perform_implicit_

Re: [C++ Patch] PR 65323

2015-03-11 Thread Jason Merrill
On 03/06/2015 03:36 AM, Paolo Carlini wrote: this is a regression about duplicate warnings with -Wzero-as-null-pointer-constant. The regression is rather old, affects 4_8-branch too, and started when check_default_argument got a perform_implicit_conversion_flags call which warns a first time, the

Re: [C++ Patch] PR 65323

2015-03-06 Thread Paolo Carlini
... in case, I think we can as well apply the below, a tad simpler. Also passes testing. Paolo. Index: decl.c === --- decl.c (revision 221230) +++ decl.c (working copy) @@ -11227,11 +11227,8 @@ check_defa