<div dir="ltr"><div>I thought about doing this as a standard thing, and came to the conclusion that it amounted to subjecting our users who use the PETSc compiler configuration to the procrustean bed.  It's mostly useful to those of us pushing to master and next.<br>
<br></div>- Peter<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 5, 2013 at 5:38 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> writes:<br>
<br>
> On Fri, 5 Jul 2013, Peter Brune wrote:<br>
><br>
>> -Wdeclaration-after-statement -Werror=declaration-after-statement<br>
>><br>
>> Is super useful and is now added to all my arches. :)<br>
><br>
> perhaps we should change configure defaults to use this?<br>
><br>
> diff --git a/config/BuildSystem/config/compilerOptions.py b/config/BuildSystem/config/compilerOptions.py<br>
> index dadc5fb..e372bb0 100644<br>
> --- a/config/BuildSystem/config/compilerOptions.py<br>
> +++ b/config/BuildSystem/config/compilerOptions.py<br>
> @@ -18,7 +18,7 @@ class CompilerOptions(config.base.Configure):<br>
>      # GNU gcc<br>
>      if config.setCompilers.Configure.isGNU(compiler) or config.setCompilers.Configure.isClang(compiler):<br>
>        if bopt == '':<br>
> -        flags.extend(['-Wall', '-Wwrite-strings', '-Wno-strict-aliasing','-Wno-unknown-pragmas'])<br>
> +        flags.extend(['-Wall', '-Wwrite-strings', '-Wno-strict-aliasing','-Wno-unknown-pragmas','-Wdeclaration-after-statement','-Werror=declaration-after-statement'])<br>

<br>
</div>I would probably skip the -Werror=... part and just recommend that<br>
developers use CFLAGS=-Werror (treat all warnings as errors).<br>
</blockquote></div><br></div>