Re: [Ada] Use "[warning enabled by default]" for default warnings

2014-02-09 Thread Robert Dewar
On 2/9/2014 3:10 PM, Richard Sandiford wrote: Which testsuite do you mean? I did test this with Ada enabled and there were no regressions. If you mean an external testsuite then I certainly don't mind holding off the Ada part. I hope the non-Ada part could still go in without it though. I m

Re: [Ada] Use "[warning enabled by default]" for default warnings

2014-02-09 Thread Richard Sandiford
Robert Dewar writes: > On 2/9/2014 3:03 PM, Richard Sandiford wrote: >> This switches Ada from using [enabled by default] to [warning enabled >> by default] for consistency with: >> >>http://gcc.gnu.org/ml/gcc-patches/2014-02/msg00549.html >> >> Tested on x86_64-linux-gnu. OK if the above pat

Re: [Ada] Use "[warning enabled by default]" for default warnings

2014-02-09 Thread Arnaud Charlet
> This switches Ada from using [enabled by default] to [warning enabled > by default] for consistency with: > > http://gcc.gnu.org/ml/gcc-patches/2014-02/msg00549.html > > Tested on x86_64-linux-gnu. OK if the above patch goes in? As I just mentioned, this isn't OK at first sight. Arno

Re: [Ada] Use "[warning enabled by default]" for default warnings

2014-02-09 Thread Robert Dewar
On 2/9/2014 3:03 PM, Richard Sandiford wrote: This switches Ada from using [enabled by default] to [warning enabled by default] for consistency with: http://gcc.gnu.org/ml/gcc-patches/2014-02/msg00549.html Tested on x86_64-linux-gnu. OK if the above patch goes in? I would say hold off on

[Ada] Use "[warning enabled by default]" for default warnings

2014-02-09 Thread Richard Sandiford
This switches Ada from using [enabled by default] to [warning enabled by default] for consistency with: http://gcc.gnu.org/ml/gcc-patches/2014-02/msg00549.html Tested on x86_64-linux-gnu. OK if the above patch goes in? Thanks, Richard gcc/ada/ * erroutc.adb (Output_Msg_Text): Use "[